HBASE-28529 Use ZKClientConfig instead of system properties when sett…

…ing zookeeper configurations

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 44s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 44s
master passed

+1 :green_heart:
compile
0m 43s
master passed

+1 :green_heart:
shadedjars
5m 43s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 26s
the patch passed

+1 :green_heart:
compile
0m 43s
the patch passed

+1 :green_heart:
javac
0m 43s
the patch passed

+1 :green_heart:
shadedjars
5m 49s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 11s
hbase-zookeeper generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

_ Other Tests _

+1 :green_heart:
unit
2m 16s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 37s
hbase-client in the patch passed.

+1 :green_heart:
unit
0m 41s
hbase-zookeeper in the patch passed.

26m 50s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 1c704d6c00cd 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Temurin-1.8.0_352-b08

javadoc
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/artifact/yetus-jdk8-hadoop3-check/output/diff-javadoc-javadoc-hbase-zookeeper.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/testReport/

Max. process+thread count
371 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 17s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 11s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 59s
master passed

+1 :green_heart:
compile
1m 31s
master passed

+1 :green_heart:
checkstyle
0m 42s
master passed

+1 :green_heart:
spotless
0m 45s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m 42s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 46s
the patch passed

+1 :green_heart:
compile
1m 31s
the patch passed

+1 :green_heart:
javac
0m 33s
hbase-common generated 0 new + 36 unchanged - 1 fixed = 36 total (was 37)

+1 :green_heart:
javac
0m 40s
hbase-client in the patch passed.

-0 :warning:
javac
0m 18s
hbase-zookeeper generated 1 new + 46 unchanged - 0 fixed = 47 total (was 46)

+1 :green_heart:
checkstyle
0m 41s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
4m 51s
Patch does not cause any errors with Hadoop 3.3.6.

+1 :green_heart:
spotless
0m 43s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
2m  4s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 30s
The patch does not generate ASF License warnings.

27m 57s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 06fbff701735 5.4.0-172-generic #190-Ubuntu SMP Fri Feb 2 23:24:22 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Eclipse Adoptium-11.0.17+8

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/artifact/yetus-general-check/output/diff-compile-javac-hbase-zookeeper.txt

Max. process+thread count
80 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 25s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 19s
master passed

+1 :green_heart:
compile
0m 47s
master passed

+1 :green_heart:
shadedjars
5m 53s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 41s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 55s
the patch passed

+1 :green_heart:
compile
0m 47s
the patch passed

+1 :green_heart:
javac
0m 47s
the patch passed

+1 :green_heart:
shadedjars
5m 51s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 10s
hbase-zookeeper generated 4 new + 0 unchanged - 0 fixed = 4 total (was 0)

_ Other Tests _

+1 :green_heart:
unit
2m 45s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 28s
hbase-client in the patch passed.

+1 :green_heart:
unit
0m 41s
hbase-zookeeper in the patch passed.

28m 13s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux b7227e2d5fe8 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Eclipse Adoptium-11.0.17+8

javadoc
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/artifact/yetus-jdk11-hadoop3-check/output/diff-javadoc-javadoc-hbase-zookeeper.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/testReport/

Max. process+thread count
383 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 22s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 59s
master passed

+1 :green_heart:
compile
1m  7s
master passed

+1 :green_heart:
shadedjars
7m  1s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  1s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 32s
the patch passed

+1 :green_heart:
compile
1m 20s
the patch passed

+1 :green_heart:
javac
1m 20s
the patch passed

+1 :green_heart:
shadedjars
7m 24s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 21s
hbase-zookeeper generated 4 new + 0 unchanged - 0 fixed = 4 total (was 0)

_ Other Tests _

+1 :green_heart:
unit
3m 36s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 53s
hbase-client in the patch passed.

+1 :green_heart:
unit
0m 49s
hbase-zookeeper in the patch passed.

37m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 7d9884ccc817 5.4.0-174-generic #193-Ubuntu SMP Thu Mar 7 14:29:28 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Eclipse Adoptium-17.0.10+7

javadoc
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/artifact/yetus-jdk17-hadoop3-check/output/diff-javadoc-javadoc-hbase-zookeeper.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/testReport/

Max. process+thread count
389 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/1/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 25s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 20s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 15s
master passed

+1 :green_heart:
compile
5m 15s
master passed

+1 :green_heart:
checkstyle
1m 12s
master passed

+1 :green_heart:
spotless
0m 47s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
8m 53s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 59s
the patch passed

+1 :green_heart:
compile
5m  6s
the patch passed

+1 :green_heart:
javac
5m  7s
root generated 0 new + 924 unchanged - 1 fixed = 924 total (was 925)

+1 :green_heart:
checkstyle
1m  8s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
5m 39s
Patch does not cause any errors with Hadoop 3.3.6.

+1 :green_heart:
spotless
0m 44s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
9m 27s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 35s
The patch does not generate ASF License warnings.

53m 53s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile xml

uname
Linux 5d37a43133cd 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
176 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 36s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 18s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m  2s
master passed

+1 :green_heart:
compile
2m  3s
master passed

+1 :green_heart:
shadedjars
5m 17s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 43s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 46s
the patch passed

+1 :green_heart:
compile
2m  4s
the patch passed

+1 :green_heart:
javac
2m  4s
the patch passed

+1 :green_heart:
shadedjars
5m 13s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 40s
the patch passed

_ Other Tests _

-1 :x:
unit
292m 55s
root in the patch failed.

326m 15s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux f9a7ddcb5311 5.4.0-176-generic #196-Ubuntu SMP Fri Mar 22 16:46:39 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/testReport/

Max. process+thread count
5480 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 44s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 49s
master passed

+1 :green_heart:
compile
2m  2s
master passed

+1 :green_heart:
shadedjars
5m 13s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 46s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 51s
the patch passed

+1 :green_heart:
compile
2m  2s
the patch passed

+1 :green_heart:
javac
2m  2s
the patch passed

+1 :green_heart:
shadedjars
5m  9s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 46s
the patch passed

_ Other Tests _

-1 :x:
unit
317m 56s
root in the patch failed.

352m 21s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 3321eb0214ca 5.4.0-174-generic #193-Ubuntu SMP Thu Mar 7 14:29:28 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Eclipse Adoptium-17.0.10+7

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/testReport/

Max. process+thread count
7895 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 12s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 33s
master passed

+1 :green_heart:
compile
1m 43s
master passed

+1 :green_heart:
shadedjars
5m 10s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m  5s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 26s
the patch passed

+1 :green_heart:
compile
1m 47s
the patch passed

+1 :green_heart:
javac
1m 47s
the patch passed

+1 :green_heart:
shadedjars
5m  6s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m  6s
the patch passed

_ Other Tests _

-1 :x:
unit
382m 32s
root in the patch failed.

411m 53s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 025a33f9baf2 5.4.0-172-generic #190-Ubuntu SMP Fri Feb 2 23:24:22 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Temurin-1.8.0_352-b08

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/testReport/

Max. process+thread count
5149 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/2/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 30s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 21s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 37s
master passed

+1 :green_heart:
compile
5m 24s
master passed

+1 :green_heart:
checkstyle
1m 21s
master passed

+1 :green_heart:
spotless
0m 45s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
10m 45s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 27s
the patch passed

+1 :green_heart:
compile
5m 50s
the patch passed

+1 :green_heart:
javac
5m 50s
root generated 0 new + 924 unchanged - 1 fixed = 924 total (was 925)

+1 :green_heart:
checkstyle
1m 19s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  2s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
5m 56s
Patch does not cause any errors with Hadoop 3.3.6.

+1 :green_heart:
spotless
0m 48s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
11m  4s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 39s
The patch does not generate ASF License warnings.

60m 20s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile xml

uname
Linux c523e404e500 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / a34b4bc517

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
176 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/3/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 34s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 27s
master passed

+1 :green_heart:
compile
5m 29s
master passed

+1 :green_heart:
checkstyle
1m 22s
master passed

+1 :green_heart:
spotless
0m 51s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
10m 32s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 31s
the patch passed

+1 :green_heart:
compile
5m 36s
the patch passed

+1 :green_heart:
javac
5m 36s
root generated 0 new + 924 unchanged - 1 fixed = 924 total (was 925)

+1 :green_heart:
checkstyle
1m 21s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
6m  9s
Patch does not cause any errors with Hadoop 3.3.6.

+1 :green_heart:
spotless
0m 50s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
11m 28s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 37s
The patch does not generate ASF License warnings.

60m 29s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile xml

uname
Linux 292637d9535e 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / a34b4bc517

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
175 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 44s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 50s
master passed

+1 :green_heart:
compile
2m  4s
master passed

+1 :green_heart:
shadedjars
5m 12s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 47s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 47s
the patch passed

+1 :green_heart:
compile
2m  2s
the patch passed

+1 :green_heart:
javac
2m  2s
the patch passed

+1 :green_heart:
shadedjars
5m 10s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 56s
the patch passed

_ Other Tests _

-1 :x:
unit
249m  3s
root in the patch failed.

282m 16s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 4aceea7cf4a5 5.4.0-174-generic #193-Ubuntu SMP Thu Mar 7 14:29:28 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / a34b4bc517

Default Java
Eclipse Adoptium-17.0.10+7

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/testReport/

Max. process+thread count
5738 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 36s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 39s
master passed

+1 :green_heart:
compile
2m 26s
master passed

+1 :green_heart:
shadedjars
7m  1s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m 20s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 43s
the patch passed

+1 :green_heart:
compile
2m 31s
the patch passed

+1 :green_heart:
javac
2m 31s
the patch passed

+1 :green_heart:
shadedjars
6m 51s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
4m 14s
the patch passed

_ Other Tests _

-1 :x:
unit
342m 53s
root in the patch failed.

419m 27s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 908bc63ebc64 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / a34b4bc517

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/testReport/

Max. process+thread count
7511 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 49s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 20s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 27s
master passed

+1 :green_heart:
compile
2m  9s
master passed

+1 :green_heart:
shadedjars
7m 33s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 37s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 18s
the patch passed

+1 :green_heart:
compile
2m 25s
the patch passed

+1 :green_heart:
javac
2m 25s
the patch passed

+1 :green_heart:
shadedjars
7m 50s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m  5s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
492m 22s
root in the patch passed.

532m  3s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 10b3646dfacb 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / a34b4bc517

Default Java
Temurin-1.8.0_352-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/testReport/

Max. process+thread count
8481 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/4/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
docker
0m  9s
Docker failed to build run-specific yetus/hbase:tp-22629}.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/5835

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/5/console

versions
git=2.17.1

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 19s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 39s
master passed

+1 :green_heart:
compile
1m 37s
master passed

+1 :green_heart:
shadedjars
5m 36s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 53s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 26s
the patch passed

+1 :green_heart:
compile
1m 35s
the patch passed

+1 :green_heart:
javac
1m 35s
the patch passed

+1 :green_heart:
shadedjars
5m 36s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 12s
the patch passed

_ Other Tests _

-1 :x:
unit
14m 43s
root in the patch failed.

41m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 799866e66c70 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / bc37ce87b0

Default Java
Temurin-1.8.0_352-b08

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/testReport/

Max. process+thread count
1290 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 56s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m  8s
master passed

+1 :green_heart:
compile
2m  2s
master passed

+1 :green_heart:
shadedjars
5m 12s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 46s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 52s
the patch passed

+1 :green_heart:
compile
2m  8s
the patch passed

+1 :green_heart:
javac
2m  8s
the patch passed

+1 :green_heart:
shadedjars
5m 11s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 46s
the patch passed

_ Other Tests _

-1 :x:
unit
16m  5s
root in the patch failed.

45m 41s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux aed0b4b69f39 5.4.0-174-generic #193-Ubuntu SMP Thu Mar 7 14:29:28 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / bc37ce87b0

Default Java
Eclipse Adoptium-17.0.10+7

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/testReport/

Max. process+thread count
1393 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 26s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 10s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 50s
master passed

+1 :green_heart:
compile
5m  1s
master passed

+1 :green_heart:
checkstyle
1m  9s
master passed

+1 :green_heart:
spotless
0m 44s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
8m 34s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 50s
the patch passed

+1 :green_heart:
compile
5m  7s
the patch passed

+1 :green_heart:
javac
5m  7s
root generated 0 new + 924 unchanged - 1 fixed = 924 total (was 925)

+1 :green_heart:
checkstyle
1m  6s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
5m 31s
Patch does not cause any errors with Hadoop 3.3.6.

+1 :green_heart:
spotless
0m 43s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
9m 11s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 33s
The patch does not generate ASF License warnings.

51m 47s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile xml

uname
Linux 54d5f3646300 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / bc37ce87b0

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
175 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m  7s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m  9s
master passed

+1 :green_heart:
compile
2m  1s
master passed

+1 :green_heart:
shadedjars
5m 51s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 26s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 15s
the patch passed

+1 :green_heart:
compile
2m 40s
the patch passed

+1 :green_heart:
javac
2m 40s
the patch passed

+1 :green_heart:
shadedjars
6m 49s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 56s
the patch passed

_ Other Tests _

-1 :x:
unit
22m  1s
root in the patch failed.

53m 55s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 25b012dcb871 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / bc37ce87b0

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/testReport/

Max. process+thread count
1477 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/6/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 26s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 55s
master passed

+1 :green_heart:
compile
5m  7s
master passed

+1 :green_heart:
checkstyle
1m 13s
master passed

+1 :green_heart:
spotless
0m 41s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
8m 31s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 52s
the patch passed

+1 :green_heart:
compile
5m  7s
the patch passed

+1 :green_heart:
javac
5m  7s
root generated 0 new + 924 unchanged - 1 fixed = 924 total (was 925)

+1 :green_heart:
checkstyle
1m  3s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
5m 20s
Patch does not cause any errors with Hadoop 3.3.6.

-1 :x:
spotless
0m 17s
patch has 24 errors when running spotless:check, run spotless:apply to fix.

+1 :green_heart:
spotbugs
9m 20s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 34s
The patch does not generate ASF License warnings.

51m 28s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile xml

uname
Linux a20334b6af3a 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / bc37ce87b0

Default Java
Eclipse Adoptium-11.0.17+8

spotless
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/artifact/yetus-general-check/output/patch-spotless.txt

Max. process+thread count
175 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 42s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 48s
master passed

+1 :green_heart:
compile
2m 17s
master passed

+1 :green_heart:
shadedjars
6m 40s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m  3s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 54s
the patch passed

+1 :green_heart:
compile
2m  8s
the patch passed

+1 :green_heart:
javac
2m  8s
the patch passed

+1 :green_heart:
shadedjars
5m 39s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m 32s
the patch passed

_ Other Tests _

-1 :x:
unit
257m 23s
root in the patch failed.

295m  1s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux c69a920aa8bc 5.4.0-174-generic #193-Ubuntu SMP Thu Mar 7 14:29:28 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / bc37ce87b0

Default Java
Eclipse Adoptium-17.0.10+7

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/testReport/

Max. process+thread count
5819 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 29s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 59s
master passed

+1 :green_heart:
compile
2m 45s
master passed

+1 :green_heart:
shadedjars
7m 44s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m 41s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  7s
the patch passed

+1 :green_heart:
compile
2m 40s
the patch passed

+1 :green_heart:
javac
2m 40s
the patch passed

+1 :green_heart:
shadedjars
6m 57s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m  7s
the patch passed

_ Other Tests _

-1 :x:
unit
292m 15s
root in the patch failed.

334m 42s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 6982c1d9c0da 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / bc37ce87b0

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/testReport/

Max. process+thread count
5016 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 30s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 19s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 41s
master passed

+1 :green_heart:
compile
1m 40s
master passed

+1 :green_heart:
shadedjars
5m 43s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 58s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 24s
the patch passed

+1 :green_heart:
compile
1m 37s
the patch passed

+1 :green_heart:
javac
1m 37s
the patch passed

+1 :green_heart:
shadedjars
5m 40s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 57s
the patch passed

_ Other Tests _

-1 :x:
unit
362m 51s
root in the patch failed.

392m 28s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux e1f8ddc1ba8e 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / bc37ce87b0

Default Java
Temurin-1.8.0_352-b08

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/testReport/

Max. process+thread count
5661 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/7/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 13s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 21s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 44s
master passed

+1 :green_heart:
compile
4m 54s
master passed

+1 :green_heart:
checkstyle
1m  5s
master passed

+1 :green_heart:
spotless
0m 41s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
10m 27s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 18s
the patch passed

+1 :green_heart:
compile
6m 21s
the patch passed

+1 :green_heart:
javac
6m 21s
root generated 0 new + 924 unchanged - 1 fixed = 924 total (was 925)

+1 :green_heart:
checkstyle
1m 20s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  2s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
5m 32s
Patch does not cause any errors with Hadoop 3.3.6.

+1 :green_heart:
spotless
1m  7s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
14m 23s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 39s
The patch does not generate ASF License warnings.

63m 16s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile xml

uname
Linux 150efad9bfe4 5.4.0-172-generic #190-Ubuntu SMP Fri Feb 2 23:24:22 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5a404c4950

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
174 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 44s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 46s
master passed

+1 :green_heart:
compile
2m  3s
master passed

+1 :green_heart:
shadedjars
5m  9s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 47s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 53s
the patch passed

+1 :green_heart:
compile
2m  1s
the patch passed

+1 :green_heart:
javac
2m  1s
the patch passed

+1 :green_heart:
shadedjars
5m  8s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 45s
the patch passed

_ Other Tests _

-1 :x:
unit
246m 36s
root in the patch failed.

279m 42s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux<PHONE_NUMBER>9e 5.4.0-174-generic #193-Ubuntu SMP Thu Mar 7 14:29:28 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5a404c4950

Default Java
Eclipse Adoptium-17.0.10+7

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/testReport/

Max. process+thread count
5189 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 24s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 48s
master passed

+1 :green_heart:
compile
2m 19s
master passed

+1 :green_heart:
shadedjars
6m 30s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 57s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 25s
the patch passed

+1 :green_heart:
compile
2m 28s
the patch passed

+1 :green_heart:
javac
2m 28s
the patch passed

+1 :green_heart:
shadedjars
6m 40s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m  7s
the patch passed

_ Other Tests _

-1 :x:
unit
283m 22s
root in the patch failed.

321m 33s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 936c1dcbce94 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5a404c4950

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/testReport/

Max. process+thread count
5220 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 28s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 29s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 33s
master passed

+1 :green_heart:
compile
1m 35s
master passed

+1 :green_heart:
shadedjars
5m 35s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 56s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 39s
the patch passed

+1 :green_heart:
compile
2m 13s
the patch passed

+1 :green_heart:
javac
2m 13s
the patch passed

+1 :green_heart:
shadedjars
6m 40s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 48s
the patch passed

_ Other Tests _

-1 :x:
unit
497m 17s
root in the patch failed.

530m 41s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux ae66ebb2e10f 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5a404c4950

Default Java
Temurin-1.8.0_352-b08

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/testReport/

Max. process+thread count
5110 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/9/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

@wchevreuil @BukrosSzabolcs PTAL.
@anmolnar FYI.
Thanks.

@Apache9 I like the flexibility of your solution, it would solve HBASE-28464. Thanks for taking the time and doing this!

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 25s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 33s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 39s
master passed

+1 :green_heart:
compile
0m 43s
master passed

+1 :green_heart:
shadedjars
5m 45s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 27s
the patch passed

+1 :green_heart:
compile
0m 43s
the patch passed

+1 :green_heart:
javac
0m 43s
the patch passed

+1 :green_heart:
shadedjars
5m 42s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 36s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 54s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 25s
hbase-client in the patch passed.

+1 :green_heart:
unit
0m 37s
hbase-zookeeper in the patch passed.

25m 53s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/10/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 9512c3a78c0c 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / e3761baec1

Default Java
Temurin-1.8.0_352-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/10/testReport/

Max. process+thread count
367 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/10/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 37s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 19s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 49s
master passed

+1 :green_heart:
compile
0m 54s
master passed

+1 :green_heart:
shadedjars
5m 16s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 48s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 49s
the patch passed

+1 :green_heart:
compile
0m 55s
the patch passed

+1 :green_heart:
javac
0m 55s
the patch passed

+1 :green_heart:
shadedjars
5m 18s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 47s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
2m 29s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 41s
hbase-client in the patch passed.

+1 :green_heart:
unit
0m 44s
hbase-zookeeper in the patch passed.

27m 21s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/10/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5835

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 4b4df1e0789c 5.4.0-176-generic #196-Ubuntu SMP Fri Mar 22 16:46:39 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / e3761baec1

Default Java
Eclipse Adoptium-11.0.17+8

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/10/testReport/

Max. process+thread count
389 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-zookeeper U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5835/10/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.


Replace for-loop to search matching element with List.contains.

Signed-off-by: Hongyin Huo<EMAIL_ADDRESS>

Issue: https://github.com/IBM/iTestCloud/issues/34
Test: com.ibm.itest.cloud.apsportal.demo.steps.StepA02_SanityTests
Result:
Starting execution of test case 'StepA02_SanityTests' at 17:12:26
=========================================================
	- 17:12:26: start test case 'test01_CreateProject'...

================================================================================
	  -> OK (in 1mn 4.742s)
	- 17:13:31: start test case 'test02_AddAssetToProject'...
	  -> Reset button element was unavailable. Therefore, no attempt was made to reset filter.
	  -> OK (in 1mn 16.665s)
	- 17:14:48: start test case 'test99_DeleteProject'...
	  -> Jobs did not exist in project 'sanity-test-project' and therefore, no attempt was made to delete any.
	  -> Active environment runtimes did not exist in project 'sanity-test-project' and therefore, no attempt was made to delete any.

================================================================================
	  -> OK (in 18.353s)

WARNING: No debug information written due argument debug=false
**********  SUMMARY OF FAILURES **********
Unknown failures : 0	Known failures : 0


Fix Travis PR-detection for the change job

This is cleaner even if the previous approach had worked :)

PR includes the "Change detection" job, as expected. Yay.

Hooray - the new "pushed branch" job doesn't include Change Detection.


Search suffix tree implementation

Explanation of Change

Reapplying the changes from this reverted PR:

https://github.com/Expensify/App/pull/48652

This PR makes sure to fix the following regressions as well:

[x] https://github.com/Expensify/App/pull/51332#issuecomment-2432573981
[x] https://github.com/Expensify/App/issues/51471#issuecomment-2442274872
[x] ~~https://github.com/Expensify/App/issues/51123~~

(Turns out this wasn't caused by these changes)

Fixed Issues

$ https://github.com/Expensify/App/issues/46591
PROPOSAL:
Tests

Test 1:

Open the chat finder page (search icon on the start page when authenticated)
Perform searches, confirm that everything is found, make sure to also search for numbers
Search for an email address (e.g. hanno@margelo.io), make sure it's found. Then search for hannomargeloio (removing all special chars), and make sure its still found
You might want to open the production version and compare your search results with the dev version to make sure you got the similar results (see note at the top)

Test 2:

Search for the user's own email address, make sure that self DM results are at the top (if its a fresh user create a few threads)

Test 3:

Create a new group with three participants
Search for the group name, including the commas, confirm that it shows in the search results

Offline tests

Same as testing steps
QA Steps

Same as testing steps
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/57bca41b-6f56-4f5a-ad23-bf25ba9465d5

Android: mWeb Chrome

https://github.com/user-attachments/assets/2c62b027-6d00-42f3-825d-28460cdbd95f

iOS: Native

https://github.com/user-attachments/assets/1102d13d-6bdf-41f8-b319-cff930399c71

iOS: mWeb Safari

https://github.com/user-attachments/assets/94f08c09-e0c2-479c-a64f-77cf40f5c8f9

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/fabbdaac-005d-4e74-8fec-72c8dac31e16

MacOS: Desktop

https://github.com/user-attachments/assets/597b8c09-9406-4aa4-90c9-18c9a7af7b00

BUG: search result display previous search result above the current search result

login with high traffic account
search for marc
search for shawn

Expected: search for shawn should display result for shawn only
Current: it displays result for marc and shawn

screenshots

BUG: Ordering result prefer contain over startWith

screenshots

@hannojg

Thanks for testing, let me fix those!

I split out a few changes here first:

https://github.com/Expensify/App/pull/52168

Status of fixed issues:

[x] BUG: search result display previous search result above the current search result
[ ] BUG: Ordering result prefer includes over startsWith
[x] BUG: Duplicated result

Thanks for testing! On the ordering bug, i looked into it and came to the conclusion that i first need to cleanup the OptionListUtils a little, its a lot of spaghetti 🍝
How it currently works is:

(getOptions) first convert all reports and users into a different format (the SearchOption format)
(getOptions) then order all search options
(filterOptions) then filter them
(filterOptions) then apply a second ordering (i think this double ordering we can maybe get rid of in the cleanup)

All of this is happening in like 2 functions and is very coupled.
The reason why the ordering is broken with our SuffixTree is that right now it works like this:

(getOptions) first convert all reports and users into a different format (the SearchOption format)
(getOptions) then order all search options
BREAKING(FastSearch): filter options using our search tree - the search tree doesn't keep the order in-tact, so its breaking here
(FastSearch) apply the second ordering

My current approach is to cleanup the current code so the search works like this (before applying the changes from this PR):

convert all to SearchOption (unfortunately we have to do this first for multiple reasons... will see if i can make a bigger refactor so that only search results that we currently need to display will be converted which would be better for performance)
filter them
then order them at once

This way the ordering should always give the same results. Also it should be more performant to just filter the limited search results, instead of ordering everything at the start.

Great! We can wait cleanup PRs then retest.

Is this PR on HOLD for https://github.com/Expensify/App/pull/52250?

I think https://github.com/Expensify/App/pull/52250 is the only one left on cleanup PRs, then @hannojg will add some changes here to fix the ordering as I think.

Seems we are almost ready to continue now that the linked PR has been merged?

Yes, will continue now. A few more cleanups / restructures are needed before but we are getting closer!!

Next cleanup PR is available for review:

https://github.com/Expensify/App/pull/53056

Working on the final cleanup PR that will provide a clear separation between filtering and ordering here:

https://github.com/Expensify/App/pull/53371

(should be good in a few hours, will update here once done)

Alright, all cleanups have been landed - working now on fixing this one!

@ahmedGaber93 @marcaaron ready for review 😊

Bug: duplicate result on SearchRouterList
https://github.com/user-attachments/assets/d7c192c6-aa0f-4ec9-b08e-8ea1612b9c4e

The search result ordering is Different between DEV and Production, is this behavior expected?

the searchTerms looks not supported, are we intended to support it on this PR?
e.g. search by "ah ga" should return result like "ahmed gaber" (ah)med (ga)ber
https://github.com/user-attachments/assets/a1423c3e-359c-41c1-a905-8b1e761f6653

the searchTerms looks not supported, are we intended to support it on this PR?

Thats a good observation and something I've missed noticing so far. I need to think about how this can efficiently be added to the current implementation!
The other two problems you mentioned should be fixed though!

the searchTerms looks not supported, are we intended to support it on this PR?

Added support for this as well now + added a test! 😊

Raised a question here on slack about the failing eslint tests

As discussed here OptionsListUtils will be migrated separately

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.ts or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] For any bug fix or new feature in this PR, I verified that sufficient unit tests are included to prevent regressions in this flow.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Thanks for the updates, I think userToInvite have been broken in the last changes
https://github.com/user-attachments/assets/59642a7f-1bd3-444d-8708-3aaefa09499d

This should be good again for testing @ahmedGaber93


Add resource importer

Due to the special requirements from launchdarkly to use the project key and the env key, I made custom methods to import the data.

what will this look like when we use it in terraform? any special considerations we should be aware of?

what will this look like when we use it in terraform? any special considerations we should be aware of?

Yes there is one about the import syntax. I'll add those in the readme file.
But quickly an import of an environment will look like import launchdarkly_environment.my-env critical-updates-dev:dev. The first part before the ":" is the project key and the second one is the environment key.
Import of a project will stay simple. import launchdarkly_project.my-project critical-updates-dev


Figure Out International Address Form Fields

Most everything on the donation form is straight-forward, except the changes to the form that switching countries causes.
I know a Drupal module is doing all of the work, but there are way too many countries listed to try and do any custom code. On a quick search, I didn't find the best information on a great library to use.
I could try to get the field data from the same code that is used now and make that an API endpoint that returns data on user changing the country or after the page loads.
It would be interesting to see what the breakdown is for orders across countries historically.

So, on production if I choose the "United Kingdom" as a country on "/checkout", then I see the image above where the state isn't filled in and it still says the country is "USA". While you can add a country, the current Drupal code does not and therefore it ends up with the default first option in the select list.
        [ 'billTo' => array(
          'firstName' => drupal_substr(rm_amp($billing_address['first_name']), 0, 50),
          'lastName' => drupal_substr(rm_amp($billing_address['last_name']), 0, 50),
          'company' => drupal_substr(rm_amp($billing_address['organisation_name']), 0, 50),
          'address' => drupal_substr(rm_amp($billing_address['thoroughfare']), 0, 60),
          'city' => drupal_substr(rm_amp($billing_address['locality']), 0, 40),
          'state' => drupal_substr(rm_amp($billing_address['administrative_area']), 0, 40),
          'zip' => drupal_substr(rm_amp($billing_address['postal_code']), 0, 20),
          'country' => rm_amp($billing_address['country']),
          'phoneNumber' => rm_amp($contact_phone),
        )]

I can map the hardcoded fields so they read something like "Address/Thoroughfare" for the field label but the value will always match "address" in Authorize.net.

added to cu-giving tests.


NTR: [Photobacterium sp.]

Proposed Term Label
Photobacterium sp.
Proposed Associated Field(s)
organism
Definition/Description
Just received a template with many bacteria species absent from the picklist.
Definition Source
No response
Guidance
No response
Alternative labels and/or synonyms
No response
Additional Comments (not an annotation)
No response
Optional - Contributor Attribution
No response

Many thanks @gwajnberg. We are working on curating your requests. These will be available upon the next public release.

@gwajnberg This NTR has been added and can be viewed in the most recent version of the GRDI template (v8.9.8).


user_medias_gql not using the "amount" parameter correctly

Hello,
I might be wrong here, but in user_medias_gql, which is used by user_media, too, I think the paramater "amount" is not taken in consideration when setting the parameters for the request (line 433).
From line 431 to line 433 is:
variables = {
    "id": user_id,
    "first": 50,  # default amount
}

when I think is should be:
variables = {
    "id": user_id,
    "first": amount,  # default amount
}

I'm a bit ignorant on the instagram APIs, but I think if I need just 1 media this will increase substantialy the time spent doing this operation.
Thanks and keep up the good work :)

To avoid having requests with an amount too big, I think the script should divide the amount modulo 50, and use that in the 'first' parameter

It could be an idea for sure. Despite that, if you look at the code, he does already handle that though the maximum number of result Instagram returns in a single page. In this way big amount are already handled. What is not handle, in my opinion, is amounts that are < 50.
I solved this using another function in the code anyway, so it's not that big of a deal :)

These are Instagram restrictions, you can only specify <= 50:
>>> data = self.public_graphql_request({'id':<PHONE_NUMBER>, 'first': 50}, query_hash="e7e2f4da4b02303f74f0841279e52d76")
>>> len(data["user"]["edge_owner_to_timeline_media"]["edges"])
50

>>> data = self.public_graphql_request({'id':<PHONE_NUMBER>, 'first': 62}, query_hash="e7e2f4da4b02303f74f0841279e52d76")
>>> len(data["user"]["edge_owner_to_timeline_media"]["edges"])
50

>>> data = self.public_graphql_request({'id':<PHONE_NUMBER>, 'first': 22}, query_hash="e7e2f4da4b02303f74f0841279e52d76")
>>> len(data["user"]["edge_owner_to_timeline_media"]["edges"])
22


Changed the query to fix invalid cast exception when fetching course …

…title

Nice, thanks for simplifying the query by removing the unnecessary table.
There seems to be no error when merging this.

@h4ck-rOOt No problem! Thanks for creating such a wonderful tool!!


error compacting TSM files: compaction aborted: bad plan

Bug report
System info:
Influx 1.4.2 on Ubuntu Linux 16.4.
Steps to reproduce:
Unknown. It happens from time to time.
Expected behavior:
No error message in the log.
Actual behavior:
I keep getting this error message in the log from time to time (once every day or two).
error compacting TSM files: compaction aborted: bad plan:
error compacting TSM files: compaction aborted: bad plan: /media/db_volume/data/_internal/monitor/6309/000004413-000000001.tsm engine=tsm1

@hpbieker How do you have your volume mounted at /media/?  Is this a directly attached SSD, NAS, NFS, RAID, docker volume, etc..?
That error is strange in that a file appears to have disappeared from the time it was planned for a compaction.  I have seen similar issues with users trying to use NFS for the data dir in the past.
Also, can you attach your config?

@jwilder The volume under /media is an ext4 filesystem located on a volumegroup device. The volume group is stored on a set of disks that is made available from our virtual machine host (VMware vSphere). The actual disks are located in a SAN with spinning disks.
I have not actually seen the message lately.
$ cat /etc/influxdb/influxdb.conf|grep -v '^ *#\|^$'
 reporting-disabled = true
[meta]
  dir = "/media/db_volume/meta"
[data]
  dir = "/media/db_volume/data"
  wal-dir = "/media/wal_disk/wal"
  wal-fsync-delay = "0.1s"
  trace-logging-enabled = true
  max-concurrent-compactions = 8
[coordinator]
  write-timeout = "60s"
  log-queries-after = "10s"
[retention]
[shard-precreation]
[monitor]
[http]
[subscriber]
  http-timeout = "120s"
[[graphite]]
[[collectd]]
[[opentsdb]]
[[udp]]
[continuous_queries]

I suspect there is something going on with how your disks are setup.  The error is more of a warning that a TSM file was part of a compaction plan, but immediately after the file could not be found.  In this case, the compaction aborts and there is a new plan created.
If you happen to see this again, can you attach the logs for the error for several minutes before the error?  Compactions log which files they are going to work on beforehand so I'm curious if there happened to be two plans that were assigned the same file (should not happen as of #8886)


Designer doesn't work for xaml in dlls that don't reference Perspex.Desktop

That's because it tries to locate all types in the output directory. It might be worth to scan in /packages directory instead.

Migrated to https://github.com/Perspex/Perspex/issues/333


Menu items

@fzaninotto @tony-shannon

Two types of menu instead of one common:

short (Charts and Patients)
full (Patient Summary + plugins)

Custom Footer with contrast mode toggling

Results on the video:
https://drive.google.com/file/d/1OAtem9APKf8Cmt6JxvkbC50PncG3TsL2/view?usp=sharing
This PR include changes from two previous PRs:

user-panel
contrast mode

@fzaninotto @tony-shannon
Francois, I fixed all notes excluded two. In one note I described a reason of our solution. I the second one I asked a question about this note.

@tony-shannon From a UX point of view, this changing menu feels disturbing. I don't have a better solution, but I think you should investigate more on navigation and user journeys.

I accept and understand that point, thank you @fzaninotto
Our original showcase didnt use a menu for "chart" or "patients" views, as per this video, so we may revert to that for now
https://www.youtube.com/watch?v=jpfIZ_HWr3w&index=4&list=PLNxHSK29ViKLrrhdPTqbYr6XGTya4uGBv&t=2s
Your point about navigation/menus is fair, we need to consider this further
Am cc'ing @simongamester here as reminder,
thanks

@tony-shannon @fzaninotto @BogdanScherban  I've been giving this some thought lately. A few ideas:

The area to the left is context sensitive to the current level. In the SPV, we see all the sections within the patients record. It would stand to reason that at other levels there are different navigations here relavent to that level i.e. Intelligence: filter charts; MPV - filter search criteria.

The area at the top is system-wide. Displays system-wide notifications, login, back navigation. For continuity, we may need to reconsider the search box as this is really an MPV element.

The area immediately below this which shows patient information could be the conduit between levels. This isn't used on intelligence or MPV but we could devise a navigation to bridge the gap.

It's probably worthwhile exploring some of these ideas in prototype to see how they feel.


test: misc. cleanups and enhancements

Remove fmt.Println statements erroneously left in unit test.
Add output from curl command in Expect statement for easier debugging in checkEgressToWorld helper function.

Signed-off by: Ian Vernon<EMAIL_ADDRESS>

test-me-please

test-me-please


switched to relative imports in brokers/init.py due to importing …

…error when using green in a virtual environment context.
I get an import error when using green as a test runner:
...
File "[directory]/django-q-tests/venv/lib/python3.4/site-packages/django_q/tasks.py", line 21, in async
broker = options.pop('broker', get_broker())
File "[directory]/django-q-tests/venv/lib/python3.4/site-packages/django_q/brokers/init.py", line 176, in get_broker
from brokers import redis_broker
ImportError: No module named 'brokers'
Using relative imports for the modules in package brokers apparently solves this issue.
Unfortunately, I cannot confirm that the changes have no side-effects as I didn't manage to entirely run the included unit test suite. Is there a how-to and / or a dependancy list for the development setup?
Regards

Current coverage is 89.88%

Merging #101 into master will decrease coverage by -6.48% as of 5541f32

@@            master    #101   diff @@
======================================
  Files           39      39
  Stmts         2560    2560
  Branches         0       0
  Methods          0       0
======================================
- Hit           2467    2301   -166
  Partial          0       0
- Missed          93     259   +166

Review entire Coverage Diff as of 5541f32

Powered by Codecov. Updated on successful CI builds.

Ok, never mind.
Codecov seems to have issues with relative imports and after reverting the code, the CI locked up when requesting pip requirements.


Feature/populate meme to database

Saving meme data to database now works
arguments

meme: title, cloudinary_url
token: auth token

 Populate meme information to meme table in database


rav1e: use libgit2 from Nixpkgs

Description of changes
This is a native build input because it's used at build time by the "built" crate, which tries to embed git info and build timestamps into the binary (ugh).
This fixes cross-compilation from x86_64 to aarch64, because git2's build.rs tries to use -m64, which is x86_64-only.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg eval

Confirms this fixes cross build from x86_64-linux to riscv64-linux too.

@ofborg build pkgsCross.aarch64-multiplatform.rav1e