Reading Docker stats without authentication

I am also introducing metricbeat from ELK and would like to read metrics of DockerFLow HAProxy. However, HAProxy module of metricbeat seems not to have username/password to authenticate to DockerFlow proxy stats interface.
Can you please add having statistics without authentication requirement?

It's done and available from the release 1.314.


tmp files store in incorrect location

@roryk  @chapmanb  : I'm running rna-seq and see hundreds of files in the "work" directory. Probably there is some typos in code for paths.

ls work
hundreds of temp-sorted files:
temp-sort-003864-327B23C6-CHK00000000-BLK103.bin
and two sam files:
temp-core-003864-6B8B4567.sam
temp-core-003868-6B8B4567.sam

I think is featureCounts, is doing something with the bam file, don't know if it is a regular step though. Maybe one solution would it be to chdir featureCount working directory? if it doesn't do that now.

Thanks @kspham, we recently cleaned up the tempfile creation in the RNA-seq parts of bcbio-nextgen so hopefully this is straightened out now. Let us know if it is still an issue. Thanks!


[BUG] choices not working

When using choices the following error still appears:
AttributeError: 'dict' object has no attribute '_json'
the following change seems to fix it:
(command.py ln. 116)
if isinstance(self._json.get("choices"), dict):  # this errors because it is a list

to
if all(isinstance(choice, dict) for choice in self.choices):

Reproducible by me, here's the full traceback:
Traceback (most recent call last):
  File "c:\Users\hi\Desktop\Coding\Python\discord-interactions v4\YouTube-tutorials\discord-py-interactions\ep1_simple_slash_commands\reference.py", line 105, in <module>
    async def hello_choices(ctx: interactions.CommandContext, choose_wisely):
  File "C:\Users\hi\AppData\Local\Programs\Python\Python39\lib\site-packages\interactions\client.py", line 358, in decorator
    commands: List[ApplicationCommand] = command(
  File "C:\Users\hi\AppData\Local\Programs\Python\Python39\lib\site-packages\interactions\decor.py",
line 83, in command
    payload: ApplicationCommand = ApplicationCommand(
  File "C:\Users\hi\AppData\Local\Programs\Python\Python39\lib\site-packages\interactions\models\command.py", line 220, in __init__
    [Option(**option) for option in self.options] if self._json.get("options") else None
  File "C:\Users\hi\AppData\Local\Programs\Python\Python39\lib\site-packages\interactions\models\command.py", line 220, in <listcomp>
    [Option(**option) for option in self.options] if self._json.get("options") else None
  File "C:\Users\hi\AppData\Local\Programs\Python\Python39\lib\site-packages\interactions\models\command.py", line 119, in __init__
    self._json["choices"] = [choice._json for choice in self.choices]
  File "C:\Users\hi\AppData\Local\Programs\Python\Python39\lib\site-packages\interactions\models\command.py", line 119, in <listcomp>
    self._json["choices"] = [choice._json for choice in self.choices]
AttributeError: 'dict' object has no attribute '_json'

Will be addressed in the next release, thanks for bringing it to our attention


Please add a license file and a screen shot

I'd like to add this to awesome-zsh-plugins, but I prefer to add themes with screen shots in the readme so that users can see what they look like without having to install them.
Also, please add a license file, some people won't use any code that doesn't have an open source license. If you don't have one in mind, https://choosealicense.com is a good resource to help you select an appropriate one.
Thanks!

Wow you're quick! I added what you were looking for and finalized some tweaks to the theme. Feel free to include it in your list if you like!

I look for updates every day or two. This will be in today's PR


fix: correctly pass query args to GET endpoints

Previously the query args for the GET endpoints weren't correctly passed, this fixes that issue.
Fixes: https://github.com/simplyspoke/node-harvest/issues/219

Thanks for merging this @simplyspoke! Is there additional work required before this can be released?

Looks like I will need to check the automation. Hopefully later today, at least this week for sure.

:tada: This PR is included in version 2.2.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@ThisIsMissEm It should be published if you want to pull it down and ensure its working. CI was out of alignment, so had to running things locally to sort it.


Please update for Swift 4

e.g. update Package.swift

Here is a pull request:
https://github.com/IBM-Swift/Kitura-Session/pull/39

Hi @gmosx, apologies for the delay in replying here. Swift 4 support had already been added in September under<EMAIL_ADDRESS>but it wasn't tagged into a release!
On Oct 27 a change was made to make Swift 4 the default, dropping Swift 3 support which was tagged as 2.0.0.
I'm sorry your issue and PR did not get attention back then :(
Thanks for taking the time to contribute, it is appreciated!


Add support for all org.apache.shardingsphere:shardingsphere-mode-type:5.3.1 submodule

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

No.

Describe the solution you'd like
A clear and concise description of what you want to happen.

Add support for org.apache.shardingsphere:shardingsphere-standalone-mode-core:5.3.1.
Add support for org.apache.shardingsphere:shardingsphere-cluster-mode-repository-zookeeper:5.3.1.
Add support for org.apache.shardingsphere:shardingsphere-cluster-mode-repository-etcd:5.3.1.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Null.

Additional context
Add any other context or screenshots about the feature request here.

Metadata generated at https://github.com/linghengqian/graalvm-trace-metadata-smoketest/issues/1 .

Linkes to https://github.com/apache/shardingsphere/issues/21347 .

I'm working on this issue.

Fixed by https://github.com/apache/shardingsphere/pull/29000.


Unable to find OSDataAccessLibrary

Hi, when I open the project, on the OSBookReviewWepApi.csproj, it doesn't find the following path Include="..\OSDataAccessLibrary\OSDataAccessLibrary.csproj" at the line 15.
Here a screenshot :

Hi @jvondermarck
The 3 separate repositories need to be forked and cloned and then all added into a singular solution something like the attached screens shot
To use the .net MAUI you must be using VS 2022 Preview edition as .net MAUI is still in Pre-Release state. but you should be able to load the web api and data access library in the same project in VS 2022 as they are using .net 6

Oh okay thank you.
So now I've done what you did but I still get some errors :

how to install this packages ??

Are you using visual studio 2022 preview edition? If so then need to go to the installer and add mobile development to list of projects you want to work on?

@brianreville Oh I downloaded the Visual Studio 2022 Entreprise and not the Visual Studio Preview (what's the difference with the Preview?)

The preview edition is general a precursor to the General Release and some of the features are classified as test status so may not work as advertised and apps that are being done under it show not be considered ready for production unless testes on a general release version.
.net Maui project type is an example , it allows cross platform coding with a single code base, including mobile and desktop apps. Its not due for general release until hopefully April but has been pushed back previously.  Some of the code used is still classified as experimental

Oh alright thank you so much.
I think I succeed to run it !
@brianreville Can I create an issue about a new feature to change the color of the navigation bar ? :)

@jvondermarck
Ya no bother ar all

@brianreville Last question how to open the XAML designer ? It only show the code on VS :

Not too sure to be honest,  I normally do it via the code page above , it use be a option in the tools page in the options editor.
Been a while since I used it though

Alright thank you so much for everything !!

No Worries at all, if your looking for some good pointers in the n
.net world
The following I find very good on YouTube
Tim Corey (IAmTimCorey)
James Montemagno
Nick Chapsas
Gerald Versluis


Make props attribute in EventOptions optional

Getting some type errors if we define revenue but not props

Published as<EMAIL_ADDRESS>Thanks!


Slack channel for Kaldi Community from Town Hall?

Can you please provide a link to a Kaldi Community slack channel, or at least let us know the process for joining?
thanks!
-Mike

https://join.slack.com/t/kaldi2020/shared_invite/zt-hnlec9ht-3RlQZV_be3eOvnRosMEL_Q
On Thu, Sep 17, 2020 at 7:24 PM mike-clark-8192<EMAIL_ADDRESS>wrote:

Can you please provide a link to a Kaldi Community slack channel, or at
least let us know the process for joinin?
thanks!
-Mike
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jtrmal/kaldi2020/issues/7, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUKYX7FB6GOXH3NHLWI5U3SGJA4HANCNFSM4RQYEQTQ
.


MR: del & restrict tax_id, documents for employee

Changes Unknown when pulling 71d2db237707b3685bd043d4371dbcb79ecd8d6b on EdwardVoloshin-patch-3 into ** on master**.

Changes Unknown when pulling 71d2db237707b3685bd043d4371dbcb79ecd8d6b on EdwardVoloshin-patch-3 into ** on master**.

Changes Unknown when pulling fb141adba7c8571eb24b1f6f55031138941a4fbc on EdwardVoloshin-patch-3 into ** on master**.


Support deep linking in the DBT docs plugin

Description
Since the DBT docs plugin uses an iframe, it doesn't support deep linking.
For example, when browsing the docs with dbt docs serve, you can get links like https://<hostname>/view#!/model/model.foo.bar. This does not happen in this plugin.
Use case/motivation
It's useful to be able to share links to a particular model in the docs. Without support for deep linking this isn't possible.
Related issues
No response
Are you willing to submit a PR?

[ ] Yes, I am willing to submit a PR!

It seems it would be technically possible to have links to deep links in iframes: https://mrvirk.com/passing-url-parameters-to-iframe-using-javascript.html, but I wonder if this would be a good approach - since Cosmos doesn't know the actual docs that it will be exposing.
@dwreeves  @bbovenzi do you have any ideas?

@tatiana Yes! @ajsquared's issue should be seen as a sub-task of #1012.

Passing the fragment identifier from the parent to the iframe was a simple 3-line change (not sure if it's the best way to do it, but it seems fine):

The reverse-- passing the fragment identifier from the iframe to the parent may be a little trickier. I'll look into it. I'm not a Javascript / front-end person so we'll see 🤔

That was fun! I figured it out. 😄 The linking will be bi-directional. PR coming shortly.

Awesome work, @dwreeves , thanks a lot for looking into this and fixing so quickly!


HADOOP-19367. Fix setting final field value on Java 17

Description of PR
Some tests use reflection to modify the final field value, which does not work after JDK-8210522 (since JDK 12)
[ERROR] org.apache.hadoop.hdfs.TestStoragePolicyPermissionSettings.testStoragePolicyPermissionDisabled -- Time elapsed: 0.014 s <<< ERROR!
java.lang.NoSuchFieldException: modifiers
	at java.base/java.lang.Class.getDeclaredField(Class.java:2610)
	at org.apache.hadoop.hdfs.TestStoragePolicyPermissionSettings.setFSNameSystemFinalField(TestStoragePolicyPermissionSettings.java:84)
	at org.apache.hadoop.hdfs.TestStoragePolicyPermissionSettings.setStoragePolicyPermissions(TestStoragePolicyPermissionSettings.java:93)
	at org.apache.hadoop.hdfs.TestStoragePolicyPermissionSettings.testStoragePolicyPermissionDisabled(TestStoragePolicyPermissionSettings.java:147)

this PR replaces them with a new approach that works with Java 8+
How was this patch tested?
The existing tests ensure it works well for Java 8, and I tested it locally with JDK 17
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.hadoop.hdfs.TestStoragePolicyPermissionSettings
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.514 s -- in org.apache.hadoop.hdfs.TestStoragePolicyPermissionSettings
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0

For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

@anujmodi2021 Could you help review this PR? Thank you very much!

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
12m 50s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 9 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
6m 43s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
32m 20s

trunk passed

+1 :green_heart:
compile
16m 26s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
15m 11s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
checkstyle
4m 17s

trunk passed

+1 :green_heart:
mvnsite
5m  6s

trunk passed

+1 :green_heart:
javadoc
4m 31s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
4m 45s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
8m 55s

trunk passed

+1 :green_heart:
shadedclient
33m 24s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 32s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m  7s

the patch passed

+1 :green_heart:
compile
16m  2s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
16m  2s

the patch passed

+1 :green_heart:
compile
15m  3s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
javac
15m  3s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
4m 13s
/results-checkstyle-root.txt
root: The patch generated 2 new + 48 unchanged - 0 fixed = 50 total (was 48)

+1 :green_heart:
mvnsite
5m  0s

the patch passed

+1 :green_heart:
javadoc
4m 22s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
4m 36s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
9m 34s

the patch passed

+1 :green_heart:
shadedclient
33m 19s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
19m 45s

hadoop-common in the patch passed.

-1 :x:
unit
226m 13s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
unit
6m 19s

hadoop-yarn-common in the patch passed.

+1 :green_heart:
unit
2m 53s

hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
1m 12s

The patch does not generate ASF License warnings.

501m 14s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestDecommission

hadoop.hdfs.server.blockmanagement.TestBlockTokenWithShortCircuitRead

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7228/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/7228

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 11279696c13b 5.15.0-124-generic #134-Ubuntu SMP Fri Sep 27 20:20:17 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / b21ecedb3caa8973d8dbe231eba629c8b0c88a3d

Default Java
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7228/1/testReport/

Max. process+thread count
3541 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-tools/hadoop-azure U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7228/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 30s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 9 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
6m 58s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
31m 13s

trunk passed

+1 :green_heart:
compile
17m 18s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
16m 42s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
checkstyle
4m 41s

trunk passed

+1 :green_heart:
mvnsite
5m 26s

trunk passed

+1 :green_heart:
javadoc
4m  4s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
4m 42s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
8m 53s

trunk passed

+1 :green_heart:
shadedclient
35m 53s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 29s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 48s

the patch passed

+1 :green_heart:
compile
16m 12s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
16m 12s

the patch passed

+1 :green_heart:
compile
16m 53s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
javac
16m 53s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
4m 40s

the patch passed

+1 :green_heart:
mvnsite
5m 18s

the patch passed

+1 :green_heart:
javadoc
3m 59s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
4m 24s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
9m 35s

the patch passed

+1 :green_heart:
shadedclient
35m 34s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
19m 41s

hadoop-common in the patch passed.

-1 :x:
unit
229m 42s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
unit
6m 21s

hadoop-yarn-common in the patch passed.

+1 :green_heart:
unit
2m 52s

hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
1m 12s

The patch does not generate ASF License warnings.

500m 35s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestDecommissionWithBackoffMonitor

hadoop.hdfs.TestDecommission

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7228/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/7228

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 41251590f263 5.15.0-124-generic #134-Ubuntu SMP Fri Sep 27 20:20:17 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 78914b2c291a742449493f4b534ac24715b1fa37

Default Java
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7228/2/testReport/

Max. process+thread count
4259 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-tools/hadoop-azure U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7228/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 33s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 9 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
7m 10s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
31m 59s

trunk passed

+1 :green_heart:
compile
19m 20s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
16m 48s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
checkstyle
4m 40s

trunk passed

+1 :green_heart:
mvnsite
4m 50s

trunk passed

+1 :green_heart:
javadoc
4m  3s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
4m  5s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
8m 15s

trunk passed

+1 :green_heart:
shadedclient
36m 40s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 31s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 11s

the patch passed

+1 :green_heart:
compile
17m  8s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
17m  8s

the patch passed

+1 :green_heart:
compile
16m 55s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
javac
16m 55s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
4m 38s

the patch passed

+1 :green_heart:
mvnsite
4m 54s

the patch passed

+1 :green_heart:
javadoc
4m  3s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
4m 20s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
9m 18s

the patch passed

+1 :green_heart:
shadedclient
36m 38s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
20m 55s

hadoop-common in the patch passed.

-1 :x:
unit
234m 30s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
unit
6m 17s

hadoop-yarn-common in the patch passed.

+1 :green_heart:
unit
2m 51s

hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
1m 13s

The patch does not generate ASF License warnings.

510m 17s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestDecommission

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7228/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/7228

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 0d6f125858ee 5.15.0-124-generic #134-Ubuntu SMP Fri Sep 27 20:20:17 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / e5b0cd714a9ae153d979daf82a54ac390158aaa7

Default Java
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7228/3/testReport/

Max. process+thread count
3702 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-tools/hadoop-azure U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7228/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

push empty to re-trigger CI

Merged. Thanks!


Gulp & browserSync issue

I'm using browserSync instead of livereload. But the browserSync not working as per the expectation.
And if i change any thing in my controller or view file every time i need to run gulp after that only i can able to view my changes.. If i refresh the page the changes are not reflected, only gulp run only reflecting my changes..
Without gulp i cant see my changes? Why it's happening?

How are you serving the page? you have to use artisan serve --host=<IP_ADDRESS>

@flick36 ..its works perfect ...Great man !..It was work in local good..how it will be act in liver server...can u explain?

@aliceopti on a live server you would have a virtual host pointing to /public which will make it work

@aliceopti it really depends on your server bro, you might be using, a VPS, or a shared hosting, and if you are using a VPS or a dedicated server you might have nginx or Apache and depending on your hosting provider, you can do certain thing or not, and there are a lot of ways to do serve the app, i can recommend you something but maybe it doesnt fit your needs, or maybe there's a better way to do it, but yeah basically as @jadjoubran said

you would have a virtual host pointing to /public which will make it work


Duplicate Entries

I am getting duplicate entries everytime I scroll on this list. Any idea what can be causing this? I'm not the only the one experiencing this issue

This has a solution?... I have the same problem!

I am having the same problem with my project.
I have a custom ArrayAdapter with custom layout for it. The first few listItems that are shown on the screen are arranged correctly (test1, test2,test3....test15) but when i scroll down to see the rest of the list, the items get scrambled...(test4, test27, test2).
Then i created another project with standard Adapter and standard simple_list_item_1 layout with 50 items and it works perfectly. I really dont know what is the problem here. Does anyone knows the solution to this problem??

Updates?


What to do with unreferenced models that are just used through visibility

Playground

import "@cadl-lang/rest";

using Cadl.Http;

model Thing {
  @visibility("update") u?: string;
  @visibility("create") c?: string;
  inner?: Thing;
}

@route("/") @post op create(...Thing): Thing;

As part of this PR #1123, the output will now be 3 schemas ThingCreate ThingRead and new Thing. As Thing itself wasn't being represented before.

Upside of emitting Thing in this case is that you can debug a little easier by comparing ThingRead to Thing in swagger. I think this is probably a fine change.
But we could get back the old behavior relatively easily, if we want it, I think. We could do the emit all models last and skip a model that already has at least one schema. I'm just not sure we want it.

@timotheeguerin @nguerrera Can't we get the old behavior by emitting models based on usage in operations?

Old Behavior
include every view of a model for the used visibility(e.g. UserRead, UserCreate)
New Behavior

(Same) include every view of a model for the used visibility(e.g. UserRead, UserCreate)
(+) include the base model (User)
if using omit-unreachable-types: true its back to the old behavior

Solutions

What we have now is good?
Only include the base model if none of the view are already included

I now think we should do (2). Emitting the "base" model suppresses code that used to avoid suffixes if there was only ever one that was used.

try to represent as a single model (with readonly if possible)
if not generate each different model but don't include base model
unreachable models stays as they are: included unless omit-unreachable-types: true

model User {
  name: string;
  @visibility("create") password: string;
  @visibility("read") id: string;
}
op create(user: User): void;
op read(): User;

components:
  schemas:
    UserRead:
      properties:
        id: { type: string }
        name: { type: string }
    UserCreate:
      properties:
        name: { type: string }
        password: { type: string }

    # Never include non used visibility
    # UserUpdate:
    #   properties:
    #    ...

    # this schema is not referenced in operations
    User:
      properties:
        id: { type: string }
        name: { type: string }
        password: { type: string }

@timotheeguerin Can you please file an implementation issue for this?

Implementation in https://github.com/microsoft/cadl/issues/1355


feature request: add support for RFC8914 Extended DNS Errors

To better understand the root cause behind SERVFAIL responses
these additional error codes would be useful (including in metrics).
https://datatracker.ietf.org/doc/rfc8914/
https://blog.cloudflare.com/unwrap-the-servfail/

This is on our planned activities somewhere, I believe, but I am not sure in what manner.

Any more info regarding this?


Revise EventArguments to not use matches(...)

"Matches" is a convenient feature but it can be confusing because the user has to look in the event class to determine which arguments go where. I propose changing
on(CommandEvent::class)
    .args("mycommand", RIGHTS_DEV)
    .run { ... }

to
command("mycommand", RIGHTS_DEV)
    .run { ... }

'command' being a function similar to 'on', which means the user is not confused to what arguments are required. This would retain the ability to intercept a general purpose CommandEvent and the performance enhancements from #68

This has been fixed with the introduction of matchers. The old (unsafe) behaviour is still available with ‘on.match’


Iron Router Support

Hi,
I've just added Iron Router support. I've tested it with web and phone on my end. But I don't have the other devices to test on.
Cheers!

+1

@justindra can add commits to the pull request. When I merge it, I can also merge a pull request from you with the changes.
I'll probably merge this soonish, I've just been really busy lately.

+1 since meteor-router is deprecated in lieu of Iron Router.

Bump.

Hey @mystor @outluch,
Sorry for the delay. I just commited your tweaks. I think that's right anyways. Please have a check.


[BUG] Keypress variable on the macros dont work properly

🔵 Describe the bug:
In old gse, keypress and keyrelease was basically places where the things you place on them was priority so if you needed to put a skill as priority you used that place,
Examples:
/targetenemy [noharm][dead]
/petattack
Or conditions like:
/cast [mod:ctrl]
/cast [mod:alt]
The problem is simple now they arent priority at all and almost in all cases skills on keypress dont work at all its like the macro simply ignore all the lines on that variable.
🔵 To reproduce: (Steps to reproduce the behavior)
only create a keypress on any macro and you have repeated the bug
🔵 Expected behavior:
I expect to have a way to assign correctly the order of commands on the macro, have a place to put priority and spameable orders so we can magane the head, body and end of the macro without dissorder.
🔵 Desktop (please complete the following information):

OS: [Windows 10 64bit]
Game Version [retail (9.1.0)]

🔵 GSE Version:

Version: 3.0.26
Downloaded From: [Wowup]

This is a core fundamental change in GSE3.  In two they were implicit. But in 3 you may not want them on every action you may want variations on different actions.  If you don’t actively put that variable anywhere nothing is going to happen.
TimothyLuke

On 12 Jul 2021, at 7:04 am, Roberto Roeschmann @.***> wrote:

🔵 Describe the bug:
In old gse, keypress and keyrelease was basically places where the things you place on them was priority so if you needed to put a skill as priority you used that place,
Examples:
/targetenemy [noharm][dead]
/petattack
Or conditions like:
/cast [mod:ctrl]
/cast [mod:alt]
The problem is simple now they arent priority at all and almost in all cases skills on keypress dont work at all its like the macro simply ignore all the lines on that variable.
🔵 To reproduce: (Steps to reproduce the behavior)
only create a keypress on any macro and you have repeated the bug
🔵 Expected behavior:
I expect to have a way to assign correctly the order of commands on the macro, have a place to put priority and spameable orders so we can magane the head, body and end of the macro without dissorder.
🔵 Desktop (please complete the following information):
OS: [Windows 10 64bit]
Game Version [retail (9.1.0)]
🔵 GSE Version:
Version: 3.0.26
Downloaded From: [Wowup]
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

This isn’t a bug - if you don’t put the variable into an action it isn’t going to occur.

Further information: https://github.com/TimothyLuke/GnomeSequencer-Enhanced/wiki/Understanding-GSE3-from-a-GSE2-Users-Perspective

BTw KeyPress and KeyRelease have a total of 0 meaning in HSE3.  You could call them Jinglebells and HotCrossBuns.  They only get meaning based on where you put them into the action blocks.  What you put at the top becomes “KeyPress” for that action.
You don’t even have to use a variable.  If it’s just one line and never going to change you can just put that one line straight onto the action block.
The variable just makes it easier to edit all at once.
TimothyLuke

On 12 Jul 2021, at 7:04 am, Roberto Roeschmann @.***> wrote:

🔵 Describe the bug:
In old gse, keypress and keyrelease was basically places where the things you place on them was priority so if you needed to put a skill as priority you used that place,
Examples:
/targetenemy [noharm][dead]
/petattack
Or conditions like:
/cast [mod:ctrl]
/cast [mod:alt]
The problem is simple now they arent priority at all and almost in all cases skills on keypress dont work at all its like the macro simply ignore all the lines on that variable.
🔵 To reproduce: (Steps to reproduce the behavior)
only create a keypress on any macro and you have repeated the bug
🔵 Expected behavior:
I expect to have a way to assign correctly the order of commands on the macro, have a place to put priority and spameable orders so we can magane the head, body and end of the macro without dissorder.
🔵 Desktop (please complete the following information):
OS: [Windows 10 64bit]
Game Version [retail (9.1.0)]
🔵 GSE Version:
Version: 3.0.26
Downloaded From: [Wowup]
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


linux_mount_label integration test

Created linux_mount_label integration test.
Referred this linux_mount_label test.
https://github.com/opencontainers/runtime-tools/tree/master/validation/linux_mount_label
#361

Hi @utam0k,
Could you take a look?
Though I think I've implemented this PR same as the integration test of runtime-tools, got an error below on my codespace environment. I've tried to fix it, but couldn't find how to fix it
# Start group linux_mount_label
1 / 1 : linux_mount_label : not ok
	container stderr was not empty, found : ERROR libcontainer::process::container_init_process: failed to mount path as masked using tempfs path="/proc/acpi" err=Nix(EINVAL)
ERROR libcontainer::process::container_init_process: failed to set masked path err=MountPathMasked(Nix(EINVAL)) path="/proc/acpi"
ERROR libcontainer::process::container_intermediate_process: failed to initialize container process: failed to mount path as masked
ERROR libcontainer::process::container_main_process: failed to wait for init ready: failed to receive. "waiting for init ready". BrokenChannel
ERROR libcontainer::container::builder_impl: failed to run container process err=Channel(ReceiveError { msg: "waiting for init ready", source: BrokenChannel })
ERROR youki: error in executing command: failed to receive. "waiting for init ready". BrokenChannel

Caused by:
    channel connection broken
Error: failed to receive. "waiting for init ready". BrokenChannel

Caused by:
    channel connection broken

# End group linux_mount_label

/label kind/test

@Gekko0114 It seems there is some issue with the /proc/acpi path due to which we are getting EINVAL error. Can  you check this path, its perms and related stuff? Thanks!

@Gekko0114 May I ask you to commit with your sign?
https://github.com/containers/youki/pull/2688/checks?check_run_id=21940370118

Codecov Report

Merging #2688 (1940783) into main (919ff4a) will increase coverage by 0.00%.
Report is 2 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2688   +/-   ##
=======================================
  Coverage   65.40%   65.41%
=======================================
  Files         133      133
  Lines       16942    16942
=======================================
+ Hits        11081    11082    +1
+ Misses       5861     5860    -1

Hi @YJDoc2 @utam0k
Though I have some questions, I created a PR and passed a test case. Could you review it?

Though I have some questions, I created a PR and passed a test case. Could you review it?

Hey, thanks! I'll take a look and answer the questions :+1:

🤦♂Our implementation is wrong.

@saschagrunert Do you know any good crate for SELinux without libselinux? I don't like adding a dependency.

@saschagrunert Do you know any good crate for SELinux without libselinux? I don't like adding a dependency.

Which functionality do you need? I assume all of them require libselinux somehow.

Which functionality do you need? I assume all of them require libselinux somehow.

I want a crate like opencontainers/selinux. It doesn't need libselinux, does it?
https://github.com/opencontainers/selinux
It seems we need following features:

label.SetFileLabel https://github.com/opencontainers/runc/blob/d5e4c33001d74176222fe8f48a323f3e8ad89999/libcontainer/rootfs_linux.go#L536
https://github.com/opencontainers/runc/blob/d5e4c33001d74176222fe8f48a323f3e8ad89999/libcontainer/rootfs_linux.go#L667-L672

			if err := label.Validate(m.Relabel); err != nil {
				return err
			}
			shared := label.IsShared(m.Relabel)
			if err := label.Relabel(m.Source, mountLabel, shared); err != nil {
				return err
			}

label.FormatMountLabel https://github.com/opencontainers/runc/blob/d5e4c33001d74176222fe8f48a323f3e8ad89999/libcontainer/rootfs_linux.go#L1247

@utam0k that does not look like much code, do we want to create a module here and then consider moving it into a new crate?

Yes, I think something like oci-spec-rs would be good. How about first implementing it in this repository to some extent, and then taking it to new repository in containers org? Maybe it can be used for other repositories/projects than youki?

Yes, I think something like oci-spec-rs would be good. How about first implementing it in this repository to some extent, and then taking it to new repository in containers org? Maybe it can be used for other repositories/projects than youki?

Yes this sounds like a good idea. :+1:

How about first implementing it in this repository to some extent,
... Maybe it can be used for other repositories/projects than youki?

After seeing the go implementation code, I was thinking along the same lines. Even if not a complete API, I think we can port the functions we need into youki, and use them :+1:
I can take a try at a basic initial implementation around the weekend, or if someone else want to try before that, let me know if you need any help!

@Gekko0114 Are you interested in this? If possible, I want you to give it a challenge with @YJDoc2 as a mentor and @saschagrunert as an advisor.

Yes, I think something like oci-spec-rs would be good. How about first implementing it in this repository to some extent, and then taking it to new repository in containers org? Maybe it can be used for other repositories/projects than youki?

@utam0k
Thanks for asking me! Yes, I would like to work on it. I will start working on this issue from this weekend.
Also I will create a new issue for this.

Thanks for asking me! Yes, I would like to work on it. I will start working on this issue from this weekend.
Also I will create a new issue for this.

That's great! Feel free to ping me on github or discord if you need any help :)


Allow JRE testing for other testsuite aside from JCK

So basically what the problem is, is that we are unable to run JRE testing with anything but JCK, since JAVA and JAVAC command variables are only defined and propagated from the jck.mk...
if we want to expand JRE testing on other testsuites we should find a suitable place to move this declarations to.
This also targets the issue of headless testsuite https://github.com/adoptium/aqa-tests/issues/5224 , that was designed with jre testing in mind and is circumventing this in its own way. We should align this with the mechanics used elsewhere in Adoptium pipeline, in this case use the same mechanic for jre testing as used in JCK.

Hi, so here is a thought: within aqa-tests with have JenkinsfileBase groovy script that seems to be handling the basic test setup.. within this script there is a function called "setup_jck_interactives" I would split this function into setting the CUSTOMIZED_SDK_URL processing and setup of JAVA_TO_TEST that we would move from the jck/jck.mk to a new function inside the script and the other jck specific procedures that would be left inside the mentioned function. This would allow execution of the new function outside of the jck context.

for example setup_customized_sdk could be the name..

or better yet.. what would be the harm in running the "setup_jck_interactives" function in non-jck test? just renaming a few things there.. there is suspiciously small amount of code that needs to be jck specific other than name-wise in my opinion

We also support running from command line for those who do not use our Jenkins scripts, so the setup logic can not be specifically at that '2nd layer' of our 3 layer design.
The logic in jck.mk will be moved to a makefile that can then be included by all of the test group makefiles (jck.mk, openjdk.mk, system.mk, etc).  Setting USE_JRE=true will be sufficient to utilize the supplied SDK for compiling tests, but the supplied JRE for executing tests.

yea, but I thought that the goal was to not have to introduce a new mk file for functional testing... we would have to create a functional.mk (non-existent now AFAIK) to have a mk file to include it by.

https://github.com/adoptium/aqa-tests/pull/5490 so this is how you would see it smlambert

I would probably not create a commonSetup.mk file in this repo, but rather leverage a make file in the TKG repository, likely https://github.com/adoptium/TKG/blob/master/testEnv.mk.  In that way, it is available to 'vendor' test material held in repositories outside of the AQAvit repositories.

How correct is my assumption that after inserting the code into testEnv.mk, I don't have to include it anywhere and the testEnv.mk would be already executed prior to every test run?

Believe it is executed prior to every test run, but of course these changes will require 2 types of testing:

validate whether this works via command line
validate via a Grinder

2 PRs needed, one in TKG and one in aqa-tests (changing testheadlessComponents command in its playlist to use JAVA_TO_TEST as the JRE/JDK to test).
Instructions for testing locally:
https://github.com/eclipse-openj9/openj9/wiki/Reproducing-Test-Failures-Locally

smlambert how about this? still need to test it tho.. it turns out the JAVA_COMMAND and JRE_COMMAND are not yet defined when testEnv.mk is called
https://github.com/adoptium/aqa-tests/pull/5492
https://github.com/adoptium/TKG/pull/596

I will run some Grinders on our private Temurin-compliance Jenkins server to verify JCK tests can still be run, if you can do some local testing for the functional use case.


Any hooks for to modify the connection?

We have a set of legacy databases we need to access, where each database has the same schema but each for a different tenant. We would like to be abled to intercept the the connection acquire and perform "use tenantdb" on teh connection before it gets to the runner. I'm not seeing any hooks, but thought I would ask.

+1

@memelet check out https://github.com/tgriesser/knex/issues/14#issuecomment-19170597 and https://github.com/tgriesser/knex/issues/619. I am still testing by afterCreate might work for my use case.

@akras14 I don't think this will work for us. We need to 'use database' for every query, not just when the connection is added to the pool.
More featured connection pools (in JVM land: http://brettwooldridge.github.io/HikariCP/) typically provide hooks for checkout/checkin for just this purpose.

I'm planning on adding something in for this

After some review, we would also need a custom hook to run on every query.
The use case is as follows:

We have 20 different users in 1 database, all separated by schema.
We only want to maintain 10 connections to that database and pool those connections between 20 users.
We want to make sure every user can only access their schema, so we need a hook to set user role and schema before every request.

@tgriesser  I would appreciate if you could let me know if there is a way to do this already. Or if it will be possible using the change that you are planning to work on.
If you need help, I'd be happy to contribute with your guidance.

This actually answers my question about whether there's a use case for having hooks in the pool as we were discussing before, Tim. However, since the user-facing interface to this functionality will be Knex in this case, I'm not sure whether it needs to be implemented at the pool level or not. I'm guessing it might help?
I'm also guessing with all the changes happening that my old hooks patch won't be sufficient here, and we should probably just close that pull request.
Let me know if you need support from the pool2 side of things and I'll try and put something in ASAP.

For our use case, we would need it at a Knex level. Obviously it would need to run for each pooled connection, but I think configuration belongs at a higher level.
Something like this
function createConnection(SOME_CLOSURE_VALUE){
  connection = knex.initialize({
    client: 'pg',
    connection: CONNECTION_STRING,
    beforeEachCall: function(conn, done) {
      conn.query("SET search_path to " + SOME_CLOSURE_VALUE + ";", function() {
        done();
      });
    }
  });
}

I'm looking for hooks that would allow modifying the query or use db based
on values from a restapi invocation.  That is, only at that time would the
values be available.  So a pre hook defined at connection create time seems
it would not work.
On May 7, 2015 7:39 PM, "Alex"<EMAIL_ADDRESS>wrote:

For our use case, we would need it at a Knex level. Obviously it would
need to run for each pooled connection, but I think configuration belongs
at a higher level.
Something like this
function createConnection(SOME_CLOSURE_VALUE){
connection = knex.initialize({
client: 'pg',
connection: CONNECTION_STRING,
beforeEachCall: function(conn, done) {
conn.query("SET search_path to " + SOME_CLOSURE_VALUE + ";", function() {
done();
});
}
});
}
—
Reply to this email directly or view it on GitHub
https://github.com/tgriesser/knex/issues/763#issuecomment-99954314.

@memelet but you could define your api call from within the hook method, and then update the query accordingly based on the result? The query won't go through until the done() is called, so it's up to you to determine when that is?

It seems like it would be more straightforward to simply create a function that calls the REST api and then calls the database. Is there some reason you wouldn't just do that?

+1 ^ Thought of that after leaving my original comment.
But I think our use case is still valid for such hook.

The api call is incoming, the database (or query restricted) is derived
from parameters.  The express middleware could set state that the knex
callback would later use.
On May 7, 2015 8:48 PM, "Alex"<EMAIL_ADDRESS>wrote:

@memelet https://github.com/memelet but you could define your api call
from within the hook method, and then update the query accordingly based on
the result? The query won't go through until the done() is called, so it's
up to you to determine when that is?
—
Reply to this email directly or view it on GitHub
https://github.com/tgriesser/knex/issues/763#issuecomment-99975526.

@akras14 were you able solve your 'use database' scenario? I have a similar need to do 'set session' every time a connection is acquired from the pool. Thanks!

@premchelliah afterCreate is called every time after new connection is created. For this issue .withSchema may work? If that doesn't work and the number of tenants is not too high you might want to create separate knex instance for each tenant with some small pool setup.
Feature which would provide work around for all of this kind of problems, would be the one, which allows user to choose using the same connection for separate queries.

@elhigu appreciate your quick response. '.afterCreate' will not work bcoz, we need to set the session (in postgres, [set session var = 'value'])  value every time the connection is acquired from the pool and not just the first time after connection creation. '.withSchema' also will not work as we have only one schema for our use case. The last solution (separate knex instance) could work; but unfortunately, we have a large user base, and this could exhaust the resources on the db. Is there a way to achieve what you mentioned 'allows user to choose using the same connection for separate queries'? Thanks again.
BTW, this article outlines the solution we are implementing. https://blog.2ndquadrant.com/application-users-vs-row-level-security

@premchelliah there is no support for such use case currently in any public APIs. Pretty much the only way to achieve that is by overriding some knex internal implementation...

Thank you @elhigu for clarifying.


AMBARI-24966 Start Namenode failing during Move master NN wizard on non-HA cluster with custom hdfs service user

How was this patch tested?
21970 passing (31s)
48 pending

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4647/
Test PASSed.


add initial docs for 4 to 5 conversion

There is a lot more work to do on these docs but this outlines how to use existing tools for creating VIC 5 parameters for image mode.
cc @tbohn: give these a try and we can work to make them more user friendly.

I'm going to merge this now so we have it in the docs.  There will be more updates to the documentation.  @tbohn - feel free to open an issue or PR for other updates to these instructions.


Modernize project layout

This PR introduces the following changes:

[x] Remove base directory automation shell and batch scripts.
[x] Start using tox.ini for automation.
[x] Update CI/CD workflows to new tox.ini.

This PR is related to https://github.com/pyansys/ansys-templates/issues/92.
Notice this is not a Python project but still takes advantage of pre-commit andtox.ini for automation.
Vale will be implemented in a separate PR. I expect lots of warnings comming from this tool as this project has lots of doc content.

Not sure where these "docs_build" and "pre-commit checks" are comming from...

I remember protection rules for main branch were discussed some time ago. From private discussion, this is something we need to enable in the settings section.
As @RobPasMue, this condition should be spread to other projects in the ecosystem.
I'll take a look to https://github.com/jcwilson/git-hooks/blob/master/included/pre-commit-protect-branch.
We may also have the possibility to implement branches protection locally.


[build]: SourceLink public release support

We need to enable GitHub sourcelink support for public nuget package releases for neonSDK and neonKUBE so folks will be able to debug into these libraries.
https://devblogs.microsoft.com/dotnet/producing-packages-with-source-link/#deterministic-builds
This entails editing the nuget publish scripts and the Directory.build.props files, where the scripts defines the NEON_PUBLIC_SOURCELINK=true environment variable that Directory.build.props uses to enable sourcelink references to GitHub.
Here's how this will work:

-local: publish disables this entirely; we'll assume that the developer is working locally

don't set NEON_PUBLIC_SOURCELINK when the local git repo is dirty

-publicsource option: when present, NEON_PUBLIC_SOURCELINK will be set even when the local git repo is dirty.  We'll use this when we're in the process of publishing apps and we will commit all of the changes eventually

Official release scripts (as opposed to dev scripts) should error when the git repo is dirty and -publicSource isn't specified

This assumes that any related commits will be pushed to GitHub eventually.

[x] neonSDK
[x] neonFORGE
[x] neonCLOUD

DONE
This has been a long-time coming.  I believe that we're doing this entirely correctly now:

End users will be able to debug from GitHub source
Us developers will be able to do the same for nuget packages on our private repo
This will also support development using nugets from local file system repos


Bar and column charts should support other types for labels

No reason DateColumn or BooleanColumn shouldn't work...

Number and Date work now. Still need DateTime, Boolean and TimeDelta


park or protected area landuse kind from NE needs normalized value

These are all National Parks, they should get similar tags as the OSM features of the same class.

Moving to next release for load balancing.

Could you give an example of this, please? I tried looking, but I couldn't find one, and it seems the low zoom landuse areas are coming from OSM anyway (apart from the urban area polygons). For example, in tile 5/5/12, I couldn't find any non-OSM ones:
$ curl -s http://vector.dev.mapzen.com/osm/landuse/5/5/12.json | \
  jq '.features[].properties.kind' | sort | uniq -c
    225 "national_park"
     51 "nature_reserve"
    831 "protected_area"
     23 "urban area"

urban area is coming from this query in landuse. Changing what it's called should be a simple matter of changing the string there - I don't think we do any further processing to it.

Verified on dev.


Improve readability of yaml files

Allow yaml files to be specified in a new format that improves readability.
Instead of using a single array to hold all information for a file or test, split parts into file, commands, options, like they are in the cached json.
This new format is activated by adding new_spec: true to the spec.
This may make specs longer, but makes them easier to understand for those new to creating specs.
Example usage:
---
assignment: hw1

compilers:
  - &cpp 'g++ --std=c++11 $@ -o<EMAIL_ADDRESS>  - &server 'g++ --std=c++11 $@ $SERVER'

files:
  - [ hello.cpp, *server, web: true ]
  - [ hw1_questions.txt ]
  - [ Makefile ]
  - [ Dog.h ]
  - [ Dog.cpp ]
  - - tryDog.cpp
    - rm -f tryDog tryDog.o Dog.o
    - make

tests:
  - [ tryDog.cpp, ./tryDog ]

inputs:
  - prelude
  - postlude
  - sd_fun.h

becomes
---
assignment: hw1

new_spec: true

compilers:
  - &cpp 'g++ --std=c++11 $@ -o<EMAIL_ADDRESS>  - &server 'g++ --std=c++11 $@ $SERVER'

files:
  - file: hello.cpp
    commands: *server
    options:
      web: true
  - file: hw1_questions.txt
  - file: Makefile
  - file: Dog.h
  - file: Dog.cpp
  - file: tryDog.cpp
    commands:
      - rm -f trydog trydog.o Dog.o
      - make

tests:
  - file: tryDog.cpp
    commands: ./tryDog

inputs:
  - prelude
  - postlude
  - sd_fun.h

Thoughts?

Oh, I'm good with that. I'll look at the code later… today? I hope. Please ping me if I'm taking too long.


Add Rules Framework

Please use the following template when submitting issues:
Issue Template
Issue Type

[ ] Bug
[ X ] Enhancement

Description
If the issue is a bug, please also include the following:
Operating System
Please include the version number (ex: Ubuntu 16.04)
Install

[ ] setup.bash
[ ] docker

DefectDojo Version
Steps to Reproduce
Sample Scan File (if applicable)
Screenshots (optional)
Console Logs (optional)

Is there a reason this was closed? I am actually looking for something like that now...


feat: require 2fa for new user to upload as well

Expand the policy to include file upload actions.
Follows #14294
Refs #13762

The response message isn't perfect, since it was geared towards management actions, but a twine upload will show:
ERROR    HTTPError: 403 Forbidden from http://localhost/legacy/
         You must enable two factor authentication to manage other settings

See https://github.com/miketheman/warehouse/pull/2048/files for a proposed refactor to improve the error message for uploads.


UX - can claim and equip be bundled?

from claim to equipped in one button

from Loothero
Yeah, frontned could do a multicall I believe or we could include logic in the contract to auto equip if you claim an item for a slot with nothing equipped yet

It's not as easy as a simple multicall unfortunatley. But we might be able to hack a way to do this. Will investigate!


解析pdf中的图像和表格

rag-gpt支持解析pdf中的图像和表格吗？

Does rag gpt support parsing images and tables in PDFs?

This is possible with strategies such as OCR, which was previously validated by local testing.
However, the corresponding model needs to be deployed, and the requirements for the entire deployment resource will be relatively high, and it has not been published on github for the time being.

If you have specific requirements for private deployment and would like to discuss further, we can communicate regarding your needs.
Email<EMAIL_ADDRESS>WeChat: Alex_Bob_1988

We will consider integrating llama-parse for parsing PDF and other files.

the file parsing capability of llama-parse
https://github.com/gpt-open/rag-gpt/pull/70


improving up command when service already exists

https://github.com/giantswarm/cli/pull/501

RFR @ewoutp

LGTM


Question/Feature request: Cancelling build

First of all, thanks for the nice library.
I have a small toy project where I am building several projects in parallel. If the first build fails I want to cancel all other pending builds - is there a possibility to cancel a build?
Thanks!

Unfortunately there isn't a great way to do this right now (at least not that I can think of - happy for suggestions here!). Because Buildalyzer "shells out" to MSBuild we've got very limited control over the forked process. We could always try to kill it, but that's really ungraceful (though maybe that's actually what we need to do). There's also a chance that sending something like a Ctrl+C combination over standard input could do the trick.
I can give this a little thought, but may not get around to an implementation for a while.


[Question/Suggestion] Needed Crucible Heat - Metal smelting points

Suggestion
Hey! I really got curious after playing in a pack abotu your mod and after I binged watched your Tutorial Vids on Youtube, I really loved everything about it!
I just had one request in my mind which I think would be really fitting:
The heat needed for each Metal smelted in the Crucible, since each Metal has a slight different heat needed, I would love to see it reflected in this step...
Like for example that Copper needs +1.085 °C or Iron +1250 °C.
Current Behavior [Optional]
Currently, every Metal despite different melting points, all get molten at 1k.
Justification
I would love to use this for some Packs, and since Temperature matters in a lot of different Parts as well, I feel this feature might fit right in, in addition of a lil' bit immersive feel. ´^^

Hi! Glad you're enjoying the mod.
The Crucible used to work that way with different melting points in older versions (it was changed to the way it works now in Crossroads 1.5.0).
It was changed because the crucible uses heat while it's above the melting point temperature, and having that 'start-using-heat' temperature change based on the contents led to heat systems with crucibles in them being kind of unreliable, because there was this massive heat-consumption that would switch on and off at various different temperatures.


MP GO:0006491    N-glycan processing

GO:0006491    N-glycan processing
2017-01-27 | Deleted | RELATION | part of GO:0006487 (protein N-linked glycosylation)
I wondered why this is. Now soem of the "N-linked glycosylation pathway" no longer slim to glycosylation?

I think this was done because it isn't clear that ALL processing is part of glycosylation.

It sounds like that are from the def:
The conversion of N-linked glycan (N = nitrogen) structures from the initially transferred oligosaccharide to a mature form, by the actions of glycosidases and glycosyltransferases
and all my annotations are. Which parts of  N-glycan processing are not part of glycosylation, do you have an example?

PMID:17074831? @krchristie will decide.

The processing (trimming)  is part of the ER QC system, but it is still part of the broader glycosylation process i.e:
https://en.wikipedia.org/wiki/N-linked_glycosylation#Processing_of_glycan
(when people talk about the process fo glycosylation, they don't exclude these steps which are absolutely required for maturation- otherwise, we no longer have a term to refer to the entire pathway)
==
This initial trimming step is thought to act as a quality control step in the ER to monitor protein folding.
Once the protein is folded correctly, the three glucose residues are removed by glucosidase I and II. The removal of the final glucose residue signals that the glycoprotein is ready for transit from the ER to the cis-Golgi.[4] However, if the protein is not folded properly, the glucose residues are not removed and thus the glycoprotein can’t leave the endoplasmic reticulum. A chaperone protein (calnexin/calreticulin) binds to the unfolded or partially folded protein to assist protein folding.

The processing (trimming)  is part of the ER QC system, but it is still part of the broader glycosylation process i.e:
https://en.wikipedia.org/wiki/N-linked_glycosylation#Processing_of_glycan
(when people talk about the process fo glycosylation, they don't exclude these steps which are absolutely required for maturation- otherwise, we no longer have a term to refer to the entire pathway)
==
This initial trimming step is thought to act as a quality control step in the ER to monitor protein folding.
Once the protein is folded correctly, the three glucose residues are removed by glucosidase I and II. The removal of the final glucose residue signals that the glycoprotein is ready for transit from the ER to the cis-Golgi.[4] However, if the protein is not folded properly, the glucose residues are not removed and thus the glycoprotein can’t leave the endoplasmic reticulum. A chaperone protein (calnexin/calreticulin) binds to the unfolded or partially folded protein to assist protein folding.

It looks to me that the lack of a relationship between these two terms is consistent with their current definitions and positions in the graph, where the first term is described as the addition of the initial carbohydrate or derivative to a protein, while the second term is described as subsequent modifications to that initial carbohydrate.
protein N-linked glycosylation
Def: A protein glycosylation process in which a carbohydrate or carbohydrate derivative unit is added to a protein via the N4 atom of peptidyl-asparagine, the omega-N of arginine, or the N1' atom peptidyl-tryptophan.
N-glycan processing
Def: The conversion of N-linked glycan (N = nitrogen) structures from the initially transferred oligosaccharide to a mature form, by the actions of glycosidases and glycosyltransferases. The early processing steps are conserved and play roles in glycoprotein folding and trafficking.
To group these two things, I think you would need some other broader term. Perhaps the existing term 'glycoprotein metabolic process' would be appropriate (def below). To do that would mean giving 'protein N-linked glycosylation' additional is_a parentage under 'glycoprotein metabolic process'. We could also add think about adding a more specific term for 'protein N-glycan metabolic process', or perhaps for 'N-linked proteoglycan biosynthetic process', which I think we could make under 'proteoglycan biosynthetic process'.
Let me know what you think.
-K
glycoprotein metabolic process
Def: The chemical reactions and pathways involving glycoproteins, any protein that contains covalently bound glycose (i.e. monosaccharide) residues; the glycose occurs most commonly as oligosaccharide or fairly small polysaccharide but occasionally as monosaccharide.
proteoglycan biosynthetic process
Def: The chemical reactions and pathways resulting in the formation of proteoglycans, any glycoprotein in which the carbohydrate units are glycosaminoglycans.

I'm a bit confused, and not an expert. But I really thought that people who study this refer to the entire pathway as "protein glycosylation". And indeed that is still a parent because
all GO:0006491 N-glycan processing
is still part of
https://www.pombase.org/term/GO:0006486
I think this is correct?
However, if this Is true, then logically
GO:0006491 N-glycan processing
should be part_of
GO:0006487 protein N-linked glycosylation


UNKNOWN_ROW_COUNT with setAutoCommit(false) and TYPE_SCROLL_INSENSITIVE

Driver version
9.2.1.jre11
SQL Server version
Microsoft SQL Server 2014 (SP3-CU4-GDR) (KB4583462) - 12.0.6433.1 (X64)   Oct 31 2020 02:54:45   Copyright (c) Microsoft Corporation  Enterprise Edition (64-bit) on Windows NT 6.3  (Build 9600: ) (Hypervisor)
Client Operating System
Microsoft Windows Server 2012 R2 Standard
JAVA/JVM version
openjdk <IP_ADDRESS> 2020-11-04 LTS
OpenJDK Runtime Environment Corretto-<IP_ADDRESS>.1 (build <IP_ADDRESS>+12-LTS)
OpenJDK 64-Bit Server VM Corretto-<IP_ADDRESS>.1 (build <IP_ADDRESS>+12-LTS, mixed mode)
Table schema
CREATE TABLE test_table(
	[col] [varchar](3)
)

go

insert into test_table
values('abc')

go 500

create view test_view as
select * from test_table
union all
select * from test_table

Problem description

Expected behaviour: the method ResultSet.getRow(), called after ResultSet.last() (in a TYPE_SCROLL_INSENSITIVE ResultSet) retrieves the current row number if greater than 1000.
Actual behaviour: UNKNOWN_ROW_COUNT (-3) is returned
Error message/stack trace: N/A
Any other details that can be helpful: this issue occurs only if setAutoCommit(false) is called before retrieving the ResultSet.

JDBC trace logs

Reproduction code
try (Connection connection = DriverManager.getConnection("jdbc:sqlserver:...")) {
            connection.setAutoCommit(false);
            try (PreparedStatement statement = connection.prepareStatement("SELECT * FROM test_view",
                    ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_READ_ONLY);
                 ResultSet resultSet = statement.executeQuery();) {
                resultSet.last();
                System.out.println(resultSet.getRow());
            }
        }

Hello Fraquak,
I've tried to reproduce this behaviour but have not been able to.
Using your table schema and reproduction code.
I am returned 1000 by the getRow() call each time.
I've tried on versions 9.2.1 and 9.3.1 of the driver as well as adding more rows to the table and therefore view.
With and without setting auto commit false.
Are there any other conditions under which you can produce this issue?

Are there any other conditions under which you can produce this issue?

I've tested the code connecting with integratedSecurity=true.

I've tried on versions 9.2.1 and 9.3.1 of the driver as well as adding more rows to the table and therefore view.

Please, try adding 1.000 rows to the table to reproduce the issue. After the first occurrence, UNKNOWN_ROW_COUNT should be returned also if 500 rows are deleted. Maybe this problem is related to the space reserved by the table.

I've reproduced this issue also with other driver versions.
It looks like this bug has always been there, since 4.2 (the oldest one I could test).

We are closing this issue since it could not be reproduced consistently and there has been a long period of inactivity. If anyone encounters this again and needs help, please comment and we can reopen.


Errors in taxonomy ranks

Hi,
I have noticed a lot of errors in the taxonomy ranks when looking at the names. At first, I used the already trained version (4.13) when I noticed the problem. I decided to use the pr2_version_5.0.0_SSU_dada2 and train it myself, thinking the newest trained set might have give better results. However, I did not see any changes in the taxonomy ranks, as they appear to be mixed up.
This is what my results look like:

If we look as the ASV1, Dreissena (genus) is correct, but the family is wrong, it should be Dreissenidae, the order should be Myida, Bivalvia is the class (and not the order), Mollusca is the phylum (not the class) and so on. So one would think to only change the names of the ranks, but it is not possible, because:
If we look at ASV6, Dinophycea is a class (correct), Dinoflagellata is a superclass and Alveolota is a superphylum.
So the column names are incorrect, but the taxonomic ranks chosen are not even consistent. When looking back at the database, "Mollusca" and "Dinophycea" are already in the same column (one is a phylum, the other is a class), which probably means that the database is mixed up, even before doing the training part.
I am wondering how are the names used for the taxonomic rank chosen? For some ASVs, it seems to be very random. Is there a way to fix this kind of issue?
Best,

Hi Daniel,
Thank you for the quick reply.
We noticed the number of level was strange too. This is how we trained the classifier, following the tutorial from the DECIPHER package:
library(seqinr)
library(DECIPHER)
library(stringr)
library(dada2)

seqs_path = "pr2_version_5.0.0_SSU_dada2.fasta" #path to the training fasta file
seqs = readDNAStringSet(seqs_path) # read the sequences to memory

set.seed(12345)
seqs = seqs[sample(length(seqs),78000)] #sample 78000 sequences from the pr2 training set
seqs = RemoveGaps(seqs) #remove gaps in the sequence if they exist
seqs = OrientNucleotides(seqs) # to make sure training sequences are in the same orientation
set.seed(NULL)

groups <- names(seqs) # sequence names

groups = str_c("Root;",groups) # all taxa must start with "Root;"
names(seqs) = groups
groupCounts <- table(groups)
u_groups <- names(groupCounts)

# Pruning the training set (see Classify Sequences from DECIPHER package if needed)
maxGroupSize <- 10 # max sequences per label (>= 1)
remove <- logical(length(seqs))
for (i in which(groupCounts > maxGroupSize)) {
  index <- which(groups==u_groups[i])
  keep <- sample(length(index),
                 maxGroupSize)
  remove[index[-keep]] <- TRUE
}
sum(remove) # number of sequences eliminated

#Train the classifier on the training set

taxid = NULL
maxIterations <- 4 # must be >= 1
allowGroupRemoval <- FALSE
probSeqsPrev <- integer() # suspected problem sequences from prior iteration
for (i in seq_len(maxIterations)) {
  cat("Training iteration: ", i, "\n", sep="")
  # train the classifier
  trainingSet <- LearnTaxa(seqs[!remove],
                           names(seqs)[!remove],
                           taxid)
  # look for problem sequences
  probSeqs <- trainingSet$problemSequences$Index
  if (length(probSeqs)==0) {
    cat("No problem sequences remaining.\n")
    break
  } else if (length(probSeqs)==length(probSeqsPrev) &&
             all(probSeqsPrev==probSeqs)) {
    cat("Iterations converged.\n")
    break
  }
  if (i==maxIterations)
    break
  probSeqsPrev <- probSeqs
  # remove any problem sequences
  index <- which(!remove)[probSeqs]
  remove[index] <- TRUE # remove all problem sequences
  if (!allowGroupRemoval) {
    # replace any removed groups
    missing <- !(u_groups %in% groups[!remove])
    missing <- u_groups[missing]
    if (length(missing) > 0) {
      index <- index[groups[index] %in% missing]
      remove[index] <- FALSE # don't remove
    }
  }
}
sum(remove) # total number of sequences eliminated
length(probSeqs) # number of remaining problem sequences
trainingSet #summary properties of the training set
saveRDS(trainingSet,file = "pr2_version5.0.0_SSU_trained.rds")

One the training was complete, we used the database with our samples:
dna = DNAStringSet(SequencesOfInterest)
seqtab.nochim = SequencesWithoutChimeras

seqs_path = ".../path_to_trained_pr2_5.0.0.fasta"
seqs = readDNAStringSet(seqs_path) # read the sequences to memory

ids <- IdTaxa(dna, trainingSet, strand="top", processors=NULL, verbose=FALSE) # use all processors
ranks <-c("rootrank","kingdom","division","phylum","class","order","family","genus") # ranks of interest

# Convert the output object of class "Taxa" to a matrix analogous to the output from assignTaxonomy
taxid <- t(sapply(ids, function(x) {
  m <- match(ranks, x$rank)
  taxa <- x$taxon[m]
  taxa[startsWith(taxa, "unclassified_")] <- NA
  taxa
}))

colnames(taxid) <- ranks
taxid[is.na(taxid)] <- "unassigned" #replace NA with unassigned for later filtering in phyloseq

#handover to phyloseq
library(phyloseq)
ps_base<-phyloseq(otu_table(seqtab.nochim,taxa_are_rows=FALSE),tax_table(taxid))
#store sequence in refseq(ps_base)
dna<-Biostrings::DNAStringSet(taxa_names(ps_base))
names(dna)<-taxa_names(ps_base)
ps_base<-merge_phyloseq(ps_base,dna)
taxa_names(ps_base)<-paste0("ASV",seq(ntaxa(ps_base)))
ps_clean<-subset_taxa(ps_base,phylum!="unassigned")

We tried to change the ranks, but we would get errors by doing so. I hope this helps.
Best,
Mélanie

Thanks Melanie for the code.  In fact PR2 provides a DECIPHER trained file that I built:
https://github.com/pr2database/pr2database/releases/download/v5.0.0/pr2_version_5.0.0_SSU.decipher.trained.rds


No such file or directory error when installing and running

After following the provided instructions, I face the following error
[INFO] [launch]: All log files can be found below /home/matt/.ros/log/2022-09-18-16-52-18-092463-matt-linux-18511
[INFO] [launch]: Default logging verbosity is set to INFO
[ERROR] [launch]: Caught exception in launch (see debug for traceback): [Errno 2] No such file or directory: '/opt/ros/humble/share/webots_ros2_epuck/robot_launch.py'

I know for a fact that I have webots_ros2_epuck and am able to launch the simulator manually using
ros2 launch webots_ros2_epuck robot_launch.py

Opened PR #2

Hi @matthew-william-lock , I have not tested the latest webots and ros humble. There may have been some changes in both.

Too late to answer, but for others.
Hi @matthew-william-lock,
You can edit the launch file mcl/launch/mcl_launch.py
on line 24:
from
os.path.join(package_dir, 'robot_launch.py')
to
os.path.join(package_dir, 'launch/robot_launch.py')
and this will hopefully remove the error.


syscall/js: ValueOf should not panic if the underlying type(kind) is supported

What version of Go are you using (go version)?

$ go version devel +dd150176c3 Fri Jul 3 03:31:29 2020 +0000 darwin/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
darwin x64

What did you do?
package main
import "syscall/js"
func main(){
type X string
x := X("hello")
v := js.ValueOf(x)
}
What did you expect to see?
x being converted to a js value.
What did you see instead?
panic: ValueOf: invalid value
I believe this is due the fact that the string value passed on is a typed.  Therefore this is a feature/improvement request to make ValueOf accept values whose underlying type matches the accepted types.

CC @neelance.

This would require a dependency from syscall/js to reflect, which is at least something that we definitely want to avoid or maybe even impossible to do.

@neelance Have you also considered internal/reflectlite?

At first glance it seems like it wouldn't be able to turn type X string into a plain string.

I'll close this issue for now, since it is very unlikely to happen due to the necessary reflect dependency. I also think that since syscall/js is a very low-level package, there need to be intermediate layers that hide it from user code and offer a nicer interface than syscall/js ever could.


Choice of pull requests

How come some pull request get approved in less than three hours, and others are years old?

The bitter truth is that those are Mailgun internal PRs.
tl;dr;
For very long time we have not paid enough... what the heck let's be honest - any attention to most of our open source repositories. I can dive into the reasons why, but that will probably be perceived by the community like whining, so I won't. But we do want to rectify the situation. So in the nearest future we (and I in particular) will be going through all the pending PRs. That does not mean that all of them are going to be accepted, but we will be considering them. I will start with updating them  asking authors if they are still interested and will proceed with those who responds.


Issues running goapp deploy in Section 4

Had issues following the tutorial at the deployment step
$ goapp deploy --version=1 --application=appname-123456 .
...
... auth process here...
....
Authentication successful.
02:19 AM Scanning files on local disk.
Error 404: --- begin server output ---
This application does not exist (project_id=u'appname-123456'). To create an App Engine application in this project, run "gcloud beta app create" in your console.
--- end server output ---
error while running appcfg.py: exit status 1

The solution was to download gcloud cli and do a dance with.
Here's the blog post of my appengine experience

Are you authenticated with the right account?
I see this error when I have more than one account and I use the wrong one.

I migrated the workshop to use gcloud now, give it a try again


The unwanted recommendation notification keeps showing

The recommendation notification keeps showing when the extension was already installed and enabled.

/duplicate of #92153 and #92210


TINKERPOP-1293: Implement GraphFilterStrategy as a default registration for GraphComputer

https://issues.apache.org/jira/browse/TINKERPOP-1293
GraphFilterStrategy (traversal optimization strategy) will create a graph filter for the OLAP traversal if one has not already been provided. This is made possible by GraphFilterStrategy analyzing the traversal to determine which edge directions/labels are being accessed and generating a traversal that meets that criteria. Along with this, GraphFilter has been advanced more intelligence, more test cases, more JavaDoc, and more helper methods for GraphComputer providers.
GraphFilterStrategy is a default GraphComputer strategy that is deactivated by TinkerGraphComputer as edge filters are expensive in TinkerGraphComputer given that the entire graph is already loaded into memory and any filtering requires labeled edges as being filtered (or not).
mvn clean install, Spark integration tests passing, and Giraph integration tests passing thus far...
VOTE +1.

CHANGELOG
* `GraphFilter` helper methods are now more intelligent when determining edge direction/label legality.
* Added `GraphFilterStrategy` to automatically construct `GraphFilters` via traversal introspection.

UPDATE
GraphFilter and GraphFilterStrategy
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

`GraphFilter` has been significantly advanced where the determination of an edge direction/label legality is more stringent. Along with this, `GraphFilter.getLegallyPositiveEdgeLabels()` has been added as a helper method to make it easier for providers to know the space of labels being accessed by the traversal.

Note that `GraphFilterStrategy` is a default `TraversalStrategy` registered with `GraphComputer.` If `GraphFilter` is expensive for the underlying `GraphComputer` implementation, it can be deactivated accordingly as is done for `TinkerGraphComputer`.

[source,java]
----
static {
  TraversalStrategies.GlobalCache.registerStrategies(TinkerGraphComputer.class,
    TraversalStrategies.GlobalCache.getStrategies(GraphComputer.class).clone().removeStrategies(GraphFilterStrategy.class));
}
----

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Apache TinkerPop .................................. SUCCESS [5.532s]
[INFO] Apache TinkerPop :: Gremlin Shaded ................ SUCCESS [2.355s]
[INFO] Apache TinkerPop :: Gremlin Core .................. SUCCESS [44.087s]
[INFO] Apache TinkerPop :: Gremlin Test .................. SUCCESS [11.803s]
[INFO] Apache TinkerPop :: Gremlin Groovy ................ SUCCESS [40.912s]
[INFO] Apache TinkerPop :: Gremlin Groovy Test ........... SUCCESS [7.801s]
[INFO] Apache TinkerPop :: TinkerGraph Gremlin ........... SUCCESS [3:29.140s]
[INFO] Apache TinkerPop :: Gremlin Benchmark ............. SUCCESS [6.440s]
[INFO] Apache TinkerPop :: Hadoop Gremlin ................ SUCCESS [11:07.210s]
[INFO] Apache TinkerPop :: Spark Gremlin ................. SUCCESS [6:18.138s]
[INFO] Apache TinkerPop :: Giraph Gremlin ................ SUCCESS [2:43:57.096s]
[INFO] Apache TinkerPop :: Neo4j Gremlin ................. SUCCESS [1:09:07.962s]
[INFO] Apache TinkerPop :: Gremlin Driver ................ SUCCESS [12.497s]
[INFO] Apache TinkerPop :: Gremlin Server ................ SUCCESS [13:22.778s]
[INFO] Apache TinkerPop :: Gremlin Console ............... SUCCESS [1:26.498s]
[INFO] Apache TinkerPop :: Gremlin Archetype ............. SUCCESS [0.161s]
[INFO] Apache TinkerPop :: Archetype - TinkerGraph ....... SUCCESS [5.221s]
[INFO] Apache TinkerPop :: Archetype - Server ............ SUCCESS [10.874s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 4:31:17.006s
[INFO] Finished at: Tue May 10 17:55:05 MDT 2016
[INFO] Final Memory: 105M/796M
[INFO] ------------------------------------------------------------------------

VOTE: +1

Nice looking tests. ran mvn clean install - all good
VOTE +1


gCNV miscellaneous optimizations and code improvements left from PR review

[ ] thread capping in SynchronizedUnivariateSolver
[ ] faster calculation of prior copy number contribution to log likelihood in IntegerCopyNumberExpectationsCalculator
[ ] optimize CoverageModelEMWorkspace.replaceMaskedEntries using Nd4j native ops

Issue moved to broadinstitute/gatk #2931 via ZenHub


core::compiler: remove Executor trait

What does this PR try to resolve?
This PR remove the Executor trait and thus simplifies some code. I added this trait for the RLS and it was never advertised as a publicly available API. The RLS is dead now and I think this API is not the best way to achieve its goals, therefore I think it should be removed.
How should we test and review this PR?
Change is trivial refactoring, so existing tests should be fine. Eyeball review should be fine.
Additional information
r? @arlosi

The first project hasn't been touched for four years, but the other two seem to be maintained. I guess we need to keep supporting this API unless we can provide an alternative or there is really good reason not to.

:umbrella: The latest upstream changes (presumably #11430) made this pull request unmergeable. Please resolve the merge conflicts.

Just posted comments for those projects. I'll take over this PR if either they agree to this or no reply for a while.

https://github.com/lu-zero/cargo-c/issues/324
https://github.com/not-fl3/cargo-quad-apk/commit/e062b673249e21d01c172166e8398221f880a720#r115110954


Improper Unique Visitor Count?

If I compare the real-time as well as historical data, the unique visitor count seems to be improper. This is from dev's latest code.

Are you still having this issue? Not sure how the data can end up like that.

@mikecao The previous issue was solved. This is new. I am attaching the screenshots of the pageviews api;

@mikecao This issue is still there in 1.34.0

Fixed in v1.37.0.


On Android, whitelisting a path with wildcard does not work if a "/" is present

On Android platform, whitelisting everything under a path with a wildcard like https://example.com/foo/* does not seem to work properly: URLs like https://example.com/foo/bar are not considered whitelisted, and opened in the system browser.
Example:
This does NOT work as expected on Android
    <allow-navigation href="https://example.com/app/*" />

This, instead, works (but might match also https://example.com/appointment):
    <allow-navigation href="https://example.com/app*" />

On iOS, both work as expected.
I can reproduce this with the following versions:
 <plugin name="cordova-plugin-whitelist" spec="^1.3.3" />
 <engine name="ios" spec="^4.5.5" />
 <engine name="android" spec="^7.1.2" />

Correction,
it works as expected, but on Android two entries have to be created for <allow-navigation />:
<allow-navigation href="https://example.com/app" />
<allow-navigation href="https://example.com/app/*" />

Otherwise the second one only won't match https://example.com/app, and the URL won't be whitelisted. On iOS, that is not necessary.
Feel free to close the issue. I am leaving it open because, while not a bug, I think this is surprising enough to deserve a mention in the documentation.

If this is different in iOS than on Android, I would indeeed consider this a bug. Thanks for reporting @lucaong!
Could you possibly go the extra mile and implement a quick reproduction app with cordova create, put it on GitHub and post the link here? This will make it much easier for someone in the future to reproduce and then fix this problem. That would be really nice 💝

Sure @janpio
here is a repo that reproduces the issue, with instructions on how to run in the README file: https://github.com/lucaong/cordova-whitelist-issue-32
It is a fresh Cordova project, with only the cordova-plugin-whitelist installed.

We are archiving this repository following Apache Cordova's Deprecation Policy. We will not continue to work on this repository. Therefore all issues and pull requests are being closed. Thanks for your contribution.


Add access control readonly and readwrite for Carbon JMX

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format:

The carbon JMX implementation doesn't support access control. Any logging user has full access. This PR fixes that problem.
Resolves: https://github.com/wso2/product-ei/issues/5200
Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

This PR close since the current branch is 4.6.x. Please find the PR for that branch. https://github.com/wso2/carbon-kernel/pull/2721


Add skeleton to scrape raw data

r? @adi-foundry/nycodex

Looks like the test file needs to be updated?

@alexander-yu -- do you mind taking a look? This ended up being bigger than I thought it would.

Codecov Report

Merging #13 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #13   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          44     63   +19
=====================================
+ Hits           44     63   +19

Impacted Files
Coverage Δ

nycodex/db.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 77677bf...d6296bf. Read the comment docs.


Feature/available liquidity class

[x] separate out available liquidity methods to own class
[x] remove redundant watcher class config inputs
[x] add simple methods to get watcher instances for cli commands
[x] add types

this pr doesn't change any functionality, mostly moves things around

@shanefontaine going to merge this one but let me know if you any comments


[BUG] Cannot resolve './src'

Describe the bug
After a fresh install of the vue cli plugin, running any task results in a 'cannot resolve './src' ' error
To Reproduce
Run any task after installing a fresh version of vue cli.
I'm running the latest version of vue cli with vue 2 selected and vue cli chrome extension version 1.1.4
Error
Module not found: Error: Can't resolve './src' in '/Users/shash/Documents/projects/swipekit-extension-test'
ERROR in main
Module not found: Error: Can't resolve './src' in '/Users/shash/Documents/projects/swipekit-extension-test'
webpack compiled with 1 error
please complete the following information:

Node Version: [16.13.0]
OS: [Mac OS Montery]

Don't exactly know why but adding a index.js inside the src folder(if it isn't already present) will fix this issue


Cannot copy and paste SQL into console

If I visit https://www.datafusion.rs/guides/getting-started-docker/and copy this sql ...
CREATE EXTERNAL TABLE uk_cities (city VARCHAR(100), lat DOUBLE, lng DOUBLE)
STORED AS CSV WITHOUT HEADER ROW
LOCATION '/opt/datafusion/data/uk_cities.csv';

... and paste it into the console (running in Docker), I get:
DataFusion Console
datafusion> CREATE EXTERNAL TABLE uk_cities (city VARCHAR(100), lat DOUBLE, lng DOUBLE)
> TORED AS CSV WITHOUT HEADER ROW
> OCATION '/opt/datafusion/data/uk_cities.csv';
Executing query ...
Error: ParserError("Expected \'STORED AS\' clause, found Some(Identifier(\"TORED\"))")

It looks the first character after a newline gets dropped somehow.

Interesting bug. From previous issues, I'd suspect liner's usage of termion, as nothing on the datafusion side looks too odd. (I don't completely understand the difference between the paste vs typing in manually, except that the pasting is so much faster.

I wonder if it is worth trying the rustyline crate instead. I will take a look at this when I get a chance.


Can Interface InputProcessor's method be optional?

I tried to use screen handle TouchUp and TouchDown event in libgdx. When I inplement InputProcessor to Screen, I have to deal all the functions like keyup, keydown, mousemoved, scrolled, etc. Many functions from interface InputProcessor are not required for my project. So if the methods in InputProcessor are optional then it will be better I guess.

InputAdapter is the class you want


Only publish known slugs

For https://trello.com/c/8kOVKy90/89-only-allow-publication-of-hmrc-manuals-with-known-slugs-in-production
Reject publishing unknown manual slugs in production environment via use of feature flag.
Concerns:

In tests we are relying on the actual data, which makes us uncomfortable. Should we look for a solution that would allow us to stub the list of known manual slugs in the tests?
We've only tested at unit level, should we add a request spec?

Nice test coverage of the new functionality, and the feature flag config looks good :flags:
We should also prevent publishing of sections if they don't belong to a known manual - sections are published via separate requests to the API so that would need to be validated separately.
We've stubbed the manuals-to-topics CSV file in some other tests, eg topics_spec.rb, but I think we're still using the real data in some other tests because it was going to get more complex than it was worth to avoid it - I remember having a conversation with @mikejustdoit about this but can't remember the exact context. If you have an approach for stubbing that seems reasonable in this context, then go ahead.

Just pushed the updated branch. Added separate validations for manual slugs for sections and addressed remaining comments, so ready for re-review :)

@mikejustdoit @h-lame I've added a custom validator to check for inclusion of slug in known manual slugs. Let me know if you see any issues with this.

@mikejustdoit I've just added a request spec for sections to https://github.com/alphagov/hmrc-manuals-api/commit/2b56be7b4821b38a5b034d5012dcc3fd9c8cde45

Thanks @davidbasalla, this is excellent :+1:


add lille university

@st877368937 Please provide us with:

the school official website URL
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.


Explicitly set config path for DNSMasq

When DNSMasq is configured to read its settings
from a folder ('-7' or '--conf-dir' option) it only
checks that the directory exists and doesn't fail if
it's empty. It could lead to a situation when DNSMasq
is running and handles requests, but not properly
configured, so some of queries can't be resolved.

ci check this


In-memory records storage (for testing)

@bsideup forgot to ask - can you add a fast way to enable it in the container?


Rename AuraDbQueryRunner

Rename AuraDbQueryRunner to SessionQueryRunner
Reformat code

 [🍓] Enable client to create sessions against self-managed dbms


Patch 1

Dear Sam,
Here goes the little document.
I hope it's ok. Do feel free to change it as will.
Best,
Franc

Ok, can you give me more details on how to write directly on the wiki :o) i am not very experienced in it uhuh
Regarding the images they are available in http://imgur.com/a/Akp3R and http://imgur.com/0pCkIiq but the markdown didn't work so well, so i just added them. Again i don't do this very often so bear with me.

No problem at all! This whole thing is a learning experience. You can just click on the Wiki tab up top and write the same markdown that you are writing in the README. pretty nice, right?

Yes, indeed! I already did it in my repo https://github.com/FrancisBase/setup/wiki . Now the last piece of the puzzle how to properly link the imgur images.

Ok i will do this after the academy weekend, i'm sure it's a simple detail. In the meanwhile thank you very much.

There's an example of how to link the images in the README.md of this repo. It looks like this
![launch jupyter notebook](https://i.imgur.com/SFeVsT6.png)

Notice the ! at the beginning. No need to wait until after the academy, you're 90% of the way there already!

Here it is!! https://github.com/FrancisBase/setup/wiki
Thank you for your support, without it i wouldn't be so fast setting the wiki, it's nice to get it done! And i have learned in the process how to build wikis!
If you can check it out (not hurry i know you are busy!) and give me feedback. Thank you!!


Add support for hapi v8

There are couple API differences from hapi v6.x to v8.x. Would be awesome if this plugin could be updated to the latest hapi version.

/cc @Josiah

Thanks @zenorocha I'll look into this soon.
Feel free to PR if you've got time.

Any news about the hapi V8 support ?