HBASE-27853 Add client side table metrics for rpc calls and request latency.

Details: HBASE-27853

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 53s
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
3m 30s
master passed

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

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

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

+1 :green_heart:
spotbugs
2m 12s
master passed

-0 :warning:
patch
1m 39s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

-0 :warning:
javac
0m 37s
hbase-client generated 2 new + 36 unchanged - 0 fixed = 38 total (was 36)

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

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

+1 :green_heart:
hadoopcheck
12m 18s
Patch does not cause any errors with Hadoop 3.2.4 3.3.4.

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

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

_ Other Tests _

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

41m 53s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 1921ddd8dfdf 5.4.0-144-generic #161-Ubuntu SMP Fri Feb 3 14:49:04 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5d82d4f746

Default Java
Eclipse Adoptium-11.0.17+8

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

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

modules
C: hbase-client hbase-server U: .

 Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

+1 :green_heart:
unit
205m 18s
hbase-server in the patch passed.

229m 21s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux a2c17b141624 5.4.0-1101-aws #109~18.04.1-Ubuntu SMP Mon Apr 24 20:40:49 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5d82d4f746

Default Java
Temurin-1.8.0_352-b08

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 52s
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 11s
Maven dependency ordering for branch

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

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

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

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

-0 :warning:
patch
5m 33s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

-1 :x:
unit
220m  3s
hbase-server in the patch failed.

247m 14s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux b06fcc42ff8a 5.4.0-148-generic #165-Ubuntu SMP Tue Apr 18 08:53:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5d82d4f746

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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 32s
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 12s
Maven dependency ordering for branch

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

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

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

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

+1 :green_heart:
spotbugs
2m 11s
master passed

-0 :warning:
patch
1m 38s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

-0 :warning:
javac
0m 38s
hbase-client generated 3 new + 36 unchanged - 0 fixed = 39 total (was 36)

-0 :warning:
checkstyle
0m 16s
hbase-client: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

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

+1 :green_heart:
hadoopcheck
9m 10s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

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

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

_ Other Tests _

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

35m 15s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux a1d466ef8091 5.4.0-152-generic #169-Ubuntu SMP Tue Jun 6 22:23:09 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / ab4b1d884f

Default Java
Eclipse Adoptium-11.0.17+8

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/2/artifact/yetus-general-check/output/diff-compile-javac-hbase-client.txt

checkstyle
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-client.txt

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  3s
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 32s
Maven dependency ordering for branch

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

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

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

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

-0 :warning:
patch
5m 41s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

+1 :green_heart:
unit
214m 36s
hbase-server in the patch passed.

239m 59s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux bc1a1a4df2e9 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 / ab4b1d884f

Default Java
Temurin-1.8.0_352-b08

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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 35s
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 49s
master passed

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

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

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

-0 :warning:
patch
5m 46s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

+1 :green_heart:
unit
225m 34s
hbase-server in the patch passed.

251m  9s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 131236e9be24 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / ab4b1d884f

Default Java
Eclipse Adoptium-11.0.17+8

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

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

modules
C: hbase-client hbase-server U: .

 Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

 Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

_ Other Tests _

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

19m 24s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux d1706053a957 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dae078e5bc

Default Java
Eclipse Adoptium-11.0.17+8

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

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

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/3/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 39s
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 _

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

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

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

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

+1 :green_heart:
spotbugs
0m 59s
master passed

-0 :warning:
patch
1m  7s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

-0 :warning:
javac
0m 39s
hbase-client generated 2 new + 36 unchanged - 0 fixed = 38 total (was 36)

-0 :warning:
checkstyle
0m 16s
hbase-client: The patch generated 1 new + 7 unchanged - 0 fixed = 8 total (was 7)

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

+1 :green_heart:
hadoopcheck
10m  5s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

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

+1 :green_heart:
spotbugs
0m 52s
the patch passed

_ Other Tests _

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

28m 51s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux e3148074cd39 5.4.0-152-generic #169-Ubuntu SMP Tue Jun 6 22:23:09 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dae078e5bc

Default Java
Eclipse Adoptium-11.0.17+8

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/3/artifact/yetus-general-check/output/diff-compile-javac-hbase-client.txt

checkstyle
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/3/artifact/yetus-general-check/output/diff-checkstyle-hbase-client.txt

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

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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 26s
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 _

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

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

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

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

-0 :warning:
patch
5m 14s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

 Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

_ Other Tests _

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

19m 53s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 745f638e6de2 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dae078e5bc

Default Java
Eclipse Adoptium-11.0.17+8

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

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

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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 31s
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 _

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

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

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

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

+1 :green_heart:
spotbugs
0m 44s
master passed

-0 :warning:
patch
0m 51s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

-0 :warning:
javac
0m 38s
hbase-client generated 3 new + 36 unchanged - 0 fixed = 39 total (was 36)

-0 :warning:
checkstyle
0m 15s
hbase-client: The patch generated 1 new + 7 unchanged - 0 fixed = 8 total (was 7)

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

+1 :green_heart:
hadoopcheck
8m 55s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

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

+1 :green_heart:
spotbugs
0m 52s
the patch passed

_ Other Tests _

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

25m 10s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux a08c87801695 5.4.0-152-generic #169-Ubuntu SMP Tue Jun 6 22:23:09 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dae078e5bc

Default Java
Eclipse Adoptium-11.0.17+8

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/4/artifact/yetus-general-check/output/diff-compile-javac-hbase-client.txt

checkstyle
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/4/artifact/yetus-general-check/output/diff-checkstyle-hbase-client.txt

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

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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.

Thanks for making those changes. There are some warnings in the pre commit hooks (checkstyle, etc). Can you fix them?
I'm going to be out of office until sept 5, so probably won't be able to re-review this until then.  Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

 Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

_ Other Tests _

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

23m 31s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 0d076bc3ec7b 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dcc5495a76

Default Java
Eclipse Adoptium-11.0.17+8

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

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

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/5/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 32s
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 _

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

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

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

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

+1 :green_heart:
spotbugs
0m 44s
master passed

-0 :warning:
patch
0m 51s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

+1 :green_heart:
hadoopcheck
9m  8s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

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

+1 :green_heart:
spotbugs
0m 51s
the patch passed

_ Other Tests _

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

25m 29s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 59f6c436e8a7 5.4.0-152-generic #169-Ubuntu SMP Tue Jun 6 22:23:09 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dcc5495a76

Default Java
Eclipse Adoptium-11.0.17+8

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/5/artifact/yetus-general-check/output/diff-compile-javac-hbase-client.txt

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

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/5/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 26s
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 _

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

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

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

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

-0 :warning:
patch
5m 15s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

 Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

_ Other Tests _

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

22m 39s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 6c683f7f98b1 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dcc5495a76

Default Java
Eclipse Adoptium-11.0.17+8

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

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

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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 31s
Docker mode activated.

 Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

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

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

+1 :green_heart:
spotbugs
0m 54s
the patch passed

_ Other Tests _

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

25m 42s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 93033a904784 5.4.0-152-generic #169-Ubuntu SMP Tue Jun 6 22:23:09 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dcc5495a76

Default Java
Eclipse Adoptium-11.0.17+8

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

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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.

Thanks for making those changes. There are some warnings in the pre commit hooks (checkstyle, etc). Can you fix them?
I'm going to be out of office until sept 5, so probably won't be able to re-review this until then. Just fyi, I'll get back to it soon.

@bbeaudreault hi, could you continue the code review for me if you are available? Thanks!

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 39s
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
3m  2s
master passed

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

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

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

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

-0 :warning:
patch
2m  9s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

+1 :green_heart:
compile
3m 23s
the patch passed

-0 :warning:
javac
0m 44s
hbase-client generated 1 new + 36 unchanged - 0 fixed = 37 total (was 36)

-0 :warning:
javac
2m 39s
hbase-server generated 1 new + 194 unchanged - 1 fixed = 195 total (was 195)

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

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

+1 :green_heart:
hadoopcheck
9m  0s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

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

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

_ Other Tests _

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

37m 16s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 70bbb8214cfe 5.4.0-152-generic #169-Ubuntu SMP Tue Jun 6 22:23:09 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 97d512be7c

Default Java
Eclipse Adoptium-11.0.17+8

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/7/artifact/yetus-general-check/output/diff-compile-javac-hbase-client.txt

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/7/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt

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

modules
C: hbase-client hbase-server U: .

 Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

-1 :x:
unit
225m 44s
hbase-server in the patch failed.

250m  2s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux e8943902d966 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 / 97d512be7c

Default Java
Temurin-1.8.0_352-b08

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

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

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

modules
C: hbase-client hbase-server U: .

 Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

-1 :x:
unit
228m 55s
hbase-server in the patch failed.

254m 22s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 60ef3863d53b 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 97d512be7c

Default Java
Eclipse Adoptium-11.0.17+8

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

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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 31s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
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 33s
master passed

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

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

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

+1 :green_heart:
spotbugs
2m 11s
master passed

-0 :warning:
patch
1m 40s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

-0 :warning:
javac
2m 25s
hbase-server generated 1 new + 194 unchanged - 1 fixed = 195 total (was 195)

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

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

+1 :green_heart:
hadoopcheck
8m 58s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

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

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

_ Other Tests _

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

34m 43s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 04e6a26f15f0 5.4.0-152-generic #169-Ubuntu SMP Tue Jun 6 22:23:09 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 97d512be7c

Default Java
Eclipse Adoptium-11.0.17+8

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/8/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/8/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 26s
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 26s
Maven dependency ordering for branch

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

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

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

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

-0 :warning:
patch
5m 49s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

-1 :x:
unit
221m 25s
hbase-server in the patch failed.

245m 41s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 16ab2aeab71c 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 / 97d512be7c

Default Java
Temurin-1.8.0_352-b08

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/8/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/8/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  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 12s
Maven dependency ordering for branch

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

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

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

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

-0 :warning:
patch
5m 40s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

-1 :x:
unit
228m 30s
hbase-server in the patch failed.

254m  1s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 0cf4266d35cd 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 97d512be7c

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/8/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/8/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.

This is looking good! One more request.
Also, can you please squash and rebase this PR so that we can clear out the patch warning in the pre-commit hook?

done.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 40s
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 17s
Maven dependency ordering for branch

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

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

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

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

+1 :green_heart:
spotbugs
3m 11s
master passed

-0 :warning:
patch
2m 18s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

+1 :green_heart:
compile
3m 34s
the patch passed

-0 :warning:
javac
2m 52s
hbase-server generated 1 new + 194 unchanged - 1 fixed = 195 total (was 195)

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

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

+1 :green_heart:
hadoopcheck
14m  2s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

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

+1 :green_heart:
spotbugs
3m 53s
the patch passed

_ Other Tests _

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

48m 49s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 66aaf0897364 5.4.0-152-generic #169-Ubuntu SMP Tue Jun 6 22:23:09 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 97d512be7c

Default Java
Eclipse Adoptium-11.0.17+8

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/9/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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 37s
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
2m 37s
master passed

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

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

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

-0 :warning:
patch
6m 17s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ 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
0m 58s
the patch passed

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

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

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

_ Other Tests _

-1 :x:
unit
1m  3s
hbase-client in the patch failed.

-1 :x:
unit
221m  9s
hbase-server in the patch failed.

246m 32s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux b8c27d9626c1 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 / 97d512be7c

Default Java
Temurin-1.8.0_352-b08

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

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

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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 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 14s
Maven dependency ordering for branch

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

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

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

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

-0 :warning:
patch
5m 47s
Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

-1 :x:
unit
1m 10s
hbase-client in the patch failed.

-1 :x:
unit
229m 21s
hbase-server in the patch failed.

254m 24s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux fb6862a0fce4 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 97d512be7c

Default Java
Eclipse Adoptium-11.0.17+8

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

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

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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.

@zhuyaogai unit test failures look related

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 31s
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 31s
master passed

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

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

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

+1 :green_heart:
spotbugs
2m 12s
master passed

_ Patch Compile Tests _

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

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

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

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

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

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

+1 :green_heart:
hadoopcheck
9m  7s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

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

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

_ Other Tests _

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

34m 50s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 9fbe7d064269 5.4.0-152-generic #169-Ubuntu SMP Tue Jun 6 22:23:09 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 97d512be7c

Default Java
Eclipse Adoptium-11.0.17+8

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/10/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 26s
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 11s
Maven dependency ordering for branch

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

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

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

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

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

-1 :x:
unit
221m 51s
hbase-server in the patch failed.

245m 11s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux d88c05ad286c 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 / 97d512be7c

Default Java
Temurin-1.8.0_352-b08

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/10/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 28s
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 35s
master passed

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

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

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

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

-1 :x:
unit
240m 25s
hbase-server in the patch failed.

266m  5s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux cb76b0fb472e 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 97d512be7c

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/10/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/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.

@bbeaudreault hi, I found that the latest unit test failures seem to  be unrelated to me?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 35s
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 13s
Maven dependency ordering for branch

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

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

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

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

+1 :green_heart:
spotbugs
2m 24s
master passed

_ Patch Compile Tests _

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

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

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

+1 :green_heart:
javac
3m 36s
the patch passed

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

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

+1 :green_heart:
hadoopcheck
8m 59s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

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

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

_ Other Tests _

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

37m 57s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 2626403a81d7 5.4.0-152-generic #169-Ubuntu SMP Tue Jun 6 22:23:09 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 02c63cdec2

Default Java
Eclipse Adoptium-11.0.17+8

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/11/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 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 18s
Maven dependency ordering for branch

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

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

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

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

_ Patch Compile Tests _

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

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

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

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

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

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

_ Other Tests _

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

-1 :x:
unit
221m 31s
hbase-server in the patch failed.

245m 19s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 1a54ef2d41ca 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 / 02c63cdec2

Default Java
Temurin-1.8.0_352-b08

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/11/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/11/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 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 14s
Maven dependency ordering for branch

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

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

+1 :green_heart:
shadedjars
4m 45s
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 14s
Maven dependency ordering for patch

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

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

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

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

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

_ Other Tests _

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

+1 :green_heart:
unit
234m 34s
hbase-server in the patch passed.

260m 17s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 81a6e2eda14f 5.4.0-156-generic #173-Ubuntu SMP Tue Jul 11 07:25:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 02c63cdec2

Default Java
Eclipse Adoptium-11.0.17+8

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

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

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5228/11/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.

Test failure looks unrelated. LGTM

@zhuyaogai thanks for all the work here! Can you submit a backport PR for branch-2?

@zhuyaogai thanks for all the work here! Can you submit a backport PR for branch-2? I think you will need to add a bit more code for the sync client (HTable) there

@bbeaudreault Okay, but it might need to be later. By the way, which branch are you referring to when you say branch-2? Branch-2 or branch-2?

Lowercase


[🚧] Dynamically retrieve character limit for text messages.

Since TeamSpeak Client version 3.3.0, clients can send and receive 8192 characters in text messages. Servers must be updated to version 3.8.0 in order to support this.
The framework should dynamically decide which character limit to use based on the client and server information.

Since I want to have a more sophisticated version for this, I'm going to push this to >1.1.0


Delay debugger pausing

If we pause right away the debugger messages that are send are not handled because the frontend is not yet initialized

:+1:


updated readme

includes sample screen shots

You can view a sample here - https://github.com/korthjp17/silverstripe-elemental-testimonials/blob/enhancement/readmeUpdates/README.md


Adds to and alphabetizes documentation of Layer base class.

Documentation for stateful and supports_masking are new.
Not sure if alphabetizing Properties is superior to the previous ordering, which put more salient, user-focused properties at the top.


Made -v flag default (to increase transparency for kubecfg apply/delete/etc)

Currently kubecfg apply [--dry-run] and kubecfg delete aren't telling you what k8s resources are being created/updated/deleted.
We should make the -v flag default to solve this, and silence it explicitly with a flag.

This was fixed in the kubecfg repository.


Resample

Based on #369 by @cpcloud. Did some cleanups, and cleared up merge conflicts.

@cpcloud do you have any thoughts on this PR?
@jcrist can you state the limitations of this iteration of resample?  From the tests it looks like we raise nice errors when the frequency is not supported.
I wonder if @sinhrks would find this PR of interest.
I'd like to get this in quickly if possible to avoid it going stale again.

As far as I can tell, this is good to go. Works on everything I've tried it on, and errors nicely on the few cases we don't support.

Fixed the comments, this could use a review @cpcloud, @sinhrks .

Woops, just found an issue that we didn't test for. Nevermind, this still isn't good.

Using rollforward doesn't work, because it doesn't take into account how binning works for resample, and instead just increments a timestamp by an offset. I ended up creating a series from the divisions, and calling resample on that to get a new set of divisions. This also takes the other kwargs into account when determining new divisions. Ping @cpcloud, @sinhrks  for review.

@cpcloud reviewed this and gave it his +1 in conversation.
I'm very glad to see this in.

+1

This may fail with pandas master.

I've found this to generally be true with every new release of pandas since around 0.15 or so.

Not limited to dask, of course.

To be explicit.  This fails on my machine

Should we make an issue?

I think that @jcrist is looking into it now

This is perhaps a pandas issue - this errors now, where before it worked:
import pandas as pd
import numpy as np

index = pd.date_range(start='20120102', periods=24*60*3, freq='T').values
index = index[::2]
s = pd.Series(np.arange(len(index), dtype='f8'), index=pd.Index(index))
expected = s.resample('30T', how='nunique')

Traceback:
Traceback (most recent call last):
  File "test.py", line 7, in <module>
    expected = s.resample('30T', how='nunique')
  File "/Users/jcrist/Code/pandas/pandas/core/generic.py", line 3368, in resample
    return sampler.resample(self).__finalize__(self)
  File "/Users/jcrist/Code/pandas/pandas/tseries/resample.py", line 82, in resample
    rs = self._resample_timestamps()
  File "/Users/jcrist/Code/pandas/pandas/tseries/resample.py", line 287, in _resample_timestamps
    result = grouped.aggregate(self._agg_method)
  File "/Users/jcrist/Code/pandas/pandas/core/groupby.py", line 2339, in aggregate
    return getattr(self, func_or_funcs)(*args, **kwargs)
  File "/Users/jcrist/Code/pandas/pandas/core/groupby.py", line 2564, in nunique
    sorter = np.lexsort((val, ids))
ValueError: all keys need to be the same shape


Parsing of time appears to be contrary to RFC3164

RFC3164 states (at 4.1.2):

The TIMESTAMP field is the local time and is in the format of "Mmm dd hh:mm:ss" (without the quote marks)

(my emphasis). rfc3164.go has the following code
ts, err = time.Parse(tsFmt, string(sub))

The documentation for time.Parse says:

In the absence of a time zone indicator, Parse returns a time in UTC.

If the local time of the syslog origin is not UTC, this will return an erroneous time. Using the local timezone of the syslog server interpreting the information is also incorrect, as that could be different from the local timezone of the syslog client. Obviously this is really a defect in the syslog protocol and sane people would set all their local timezones (for syslog purposes) to UTC, but apparently not everyone is sane.
It would therefore be useful to be able to specify the 'local' timezone of the server to the parser; if this is set, use time.ParseInLocation() else (as currently) use Parse().
I think this should be unnecessary for RFC5424 parsing.

It would therefore be useful to be able to specify the 'local' timezone of the server to the parser; if this is set, use time.ParseInLocation() else (as currently) use Parse().

Sounds like a good idea to me. Could you please provide a patch ?
Thanks :)

See pull request #7


Failed Nautobot job does not revert database changes.

Environment

Python version: 3.8
Nautobot version: 1.1.6

Steps to Reproduce

Start a Network Importer Job that will perform changes to Nautobot assets.
Make sure an exception condition happens to make the job fail.

Expected Behavior
All changes performed before the exception is thrown should be reverted after the job fails.

Observed Behavior
OBS: Although the following involves quite a bit of NTC's network-importer, I'm afraid the root cause might be on Nautobot's side. Bare with me as I try to explain what I'm seeing:

We extend Network-Importer's NautobotInterface model with our own custom model to make sure certain tags are added to certain interfaces (that is, when those tags don't already exist)
We also extend Network-Importer's NautobotCable model to make sure we throw an exception when a Cable is seen in Nautobot, but It doesn't exist in the Network.
Nautobot's Job class calls the job.run method under an atomic transaction: https://github.com/nautobot/nautobot/blob/v1.1.6/nautobot/extras/jobs.py#L1092. Meaning if anything goes wrong with the job, it must revert any changes made during its execution.
BEFORE invoking the network-importer in Nautobot, I mede sure there was no tags assigned to a particular interface that I absolutely knew would be tagged by our custom network-importer code (the Interface mode I described above). I also made sure to create a cable connection that doesn't exist in the actual network, so the job would definitely fail.
What I expected to see was the Network Importer job successfully tagging the interface, and this happens as expected. I also expected to see the job failing with the cabling exception and message "Database changes have been reverted due to error." (from the link above). This also happens and the message saying the changes were reverted is logged.
Finally, I expected to see my interface still untagged, but it remains tagged after the job fails.  Meaning reverting the atomic operation DOES NOT happen.

I have attached 2 prints:
One showing the interface being updated with the desired tag (about 1 second before the job fails), and another showing that the job  indeed failed and the "Database changes have been reverted due to error." was printed.

So, network-importer uses the API (through pynautobot) not the ORM to make changes to the database. When calling through the API, it is making these changes outside the atomic transaction.  I don't know if it is possible to make API calls using atomic transactions. You might move the tag creation to happen after the cable connection.

@theandrelima
Are you able to share a sanitized copy of your code?

At the moment API changes are not done in a way to support atomic transactions.


Is it possible to specify target directory for new applications?

With rails, I can do
rails new appname .
to create a new application in the current directory (without creating a subdirectory).
Is this possible with rails-api as well?

Hi @stieler-it, yes you can simply use rails-api
like so:
$ rails-api new burguershop

      create
      create  README.rdoc
      create  Rakefile
      create  config.ru
....

That's not creating it in the current directory though.

@stieler-it what you looking for is actually rails-api new .

You should be able to do the same thing in both projects, please comment back if it's still an issue.


Website audio generation is giving constant Interval Server Error

I am a premium user on the fish audio website and have been unable to generate audio for over 5 hours due to the constant Internal Server Error on the site. I am unsure what is causing it, as it is not providing any additional information on the error.
Alternatively I tried calling the API, and was given a 400: Bad Request response.
Do these point to some kind of server issue or bug?
Thank you.

Should be fixed, it's caused by a new behavior introduced in envoy.


Allow the KAS to finish its graceful shutdown

Currently, the KAS has a 30 seconds TerminationGracePeriodSeconds but
its shutdown sequence takes 90 seconds, where it still takes requests
for the first 30 seconds and finishes responding to them for the next 60
seconds. This is to ensure that client requests created during a rollout
still get a response.
The current setting means that the kubelet just kills the KAS after 30
seconds, interrupting this sequence. Bump TerminationGracePeriodSeconds
to 95 seconds to allow the shutdown sequence to finish. This is visible
in the KAS log where we now see [graceful-termination] apiserver is exiting in the very end after deleting a pod.
The shutdown sequence of the KAS is described in more detail here:
https://github.com/openshift/installer/blob/master/docs/dev/kube-apiserver-health-check.md

/lgtm


Update sprockets-rails to 2.2.2

Support new bootstrap themes in
TFDSo

[#90940580]

:+1: great!  Thanks!


Change text in a span

Hi,
I have data in a table I would like to edit with demo monkey. Each row in the table is in a span. Is there a way to access this with demo monkey?
The format is:
`

CHANGE THIS TEXT

`

hi,
does
CHANGE THIS TEXT = my replacement

not work?

Hi svrnm,
Thanks for the note.
Looks like the table is built with javascript, so the code is not in the base html.
Any way to get around this?
Thanks.

Looks like the table is built with javascript, so the code is not in the base html.

that should not be an issue, because DemoMonkey is working on the final result.
To analyse this problem it would be helpful to get some more details, is there a demo monkey config you can share and maybe even a webpage where I can see the problem happening?

@laz666 any more updates on this?


Implement WinUI 3 To Flutter

I have Suggestion Please Implement WinUI 3 To Flutter Framework.

Closing as a duplicate of https://github.com/flutter/flutter/issues/46481. Please follow up on it for further updates.


Update ics package to latest version: v2.29.0

This updates the ics package to the latest version. It seems that the ics.createEvent function now takes dates as arrays instead of moment objects. 🤔
Related to https://github.com/opencollective/opencollective-api/pull/6212

@Betree :

I don't get it, there is no update to package.json and the package-lock.json update seems unrelated. Did you forget to commit them?

Ops, I think I somehow missed to commit those. I've corrected it. Thanks much for pointing it out. 👍🏽

Also, I don't see this change listed on https://github.com/adamgibbons/ics/releases, is the change documented somewhere?

Yes I didn't see either, but if you notice their readme, it seems to use the array format for dates.

Did you try the update to make sure the ICS file was properly formatted?

Yes. Here's a sample file which I've generated.
BEGIN:VCALENDAR
VERSION:2.0
CALSCALE:GREGORIAN
PRODID:adamgibbons/ics
METHOD:PUBLISH
X-PUBLISHED-TTL:PT1H
BEGIN:VEVENT
UID:49d51e21-82f4-488c-b020-0a18db01b0d9
SUMMARY:Test Agora Event
DTSTAMP:20210722T152300Z
DTSTART:19691202T000000Z
DTEND:20210622T131700Z
URL:http://localhost:3000/agora-collective/events/test-agora-event-73de3f3d
STATUS:CONFIRMED
ORGANIZER;CN=Agora<EMAIL_ADDRESS>	ve.com
BEGIN:VALARM
ACTION:AUDIO
REPEAT:2
ATTACH;VALUE=URI:Glass
TRIGGER:-PT24H
END:VALARM
BEGIN:VALARM
ACTION:AUDIO
REPEAT:2
ATTACH;VALUE=URI:Glass
TRIGGER:-PT72H
END:VALARM
END:VEVENT
END:VCALENDAR

Yes. Here's a sample file which I've generated.

Great, if these timestamps match the event dates I'm all good with this update!

Thanks, I've created an issue on that repo as well just to double check and maybe add it to the documentation; https://github.com/adamgibbons/ics/issues/190. 😸

The month is wrong. For an event in October, it's displayed September. Can that be fixed?

The month is wrong. For an event in October, it's displayed September. Can that be fixed?

@znarf : Fixed at https://github.com/opencollective/opencollective-api/pull/6462. Let me know how it looks.


Could you provide a readme document to use？

Hello ， I find this repository ，and i don't known how to use?
So,Could you provide a readme document to use？

Sure @HivenKay, I am writing the document for this package. I hope that the documentation can be completed today :smile:

Thank you very  much,and some of your classes are very helpful to me.

closed #1


Pattern matching breaks with CWD backups

Output of restic version
restic 0.17.0 compiled with go1.22.5 on linux/amd64
Problem description / Steps to reproduce
As titled; the CWD itself can be erroneously matched. This had me confused for a bit - I always make a top-level restic script and was doing a quick initial repo setup in a folder also named restic which I was moving data into.
Setup:
mkdir -p temp/repo temp/test
cd temp/test
printf "content\n" | tee file test
restic -r ../repo init

We are now in a folder named test which we would like to back up. It contains file which is desired, and test which is not.
Issue:
restic -r ../repo backup . --exclude="./test"
enter password for repository:
repository 1268c009 opened (version 2, compression level auto)
created new cache in /home/isaacelenbaas/.cache/restic
no parent snapshot found, will read all files
[0:00]          0 index files loaded
Fatal: unable to save snapshot: snapshot is empty

Expected behavior
Don't exclude the entire CWD, even though it has the same name - there was a ./ prefix on the --exclude. Using $PWD works:
restic -r ../repo backup "$PWD" --exclude="./test"
enter password for repository:
repository 1268c009 opened (version 2, compression level auto)
no parent snapshot found, will read all files
[0:00]          0 index files loaded

Files:           0 new,     0 changed,     0 unmodified
Dirs:            4 new,     0 changed,     0 unmodified
Added to the repository: 1.508 KiB (1.243 KiB stored)

processed 0 files, 0 B in 0:00
snapshot e0cbfe8e saved

Actual behavior
The current working directory was matched and excluded, making the snapshot empty.
Do you have any idea what may have caused this?
The repository location does not matter. I would have guessed that restic moves to the backup directory's parent at some point without considering how that affects filepath.Match, but using $PWD does not exhibit the same behavior.

I think your steps to reproduce are broken. With the given steps, all files in the backup source folder are excluded, such that the snapshot is empty - just as requested. What yields behavior that matches the issue title are the following steps:
mkdir -p temp/temp
cd temp/temp
echo "content" > test
restic -r ../repo backup . --exclude="./temp"

That particular behavior is a duplicate of https://github.com/restic/restic/issues/2424 . An exclude path ./test is normalized to test before being used for excluding. Having a dedicated exclude syntax for relative backups is tracked in https://github.com/restic/restic/issues/3007 .

I missed the "Patterns. . . tested against the full path of a file/dir to be saved, even if restic is passed a relative path to save" note which would show that the ideal way to do this would be --exclude="/temp".
However, ./ stripping does not appear to be documented and not every folder is .; doing so doesn't make sense to me. Both the "Skip creating snapshots if unchanged" and "Including Files" sections recommend relative paths, and --exclude not working with them is an odd and in this case dangerous difference. I gave a path to a file and now all files and folders anywhere named test have been excluded which I might have missed if that didn't describe a parent directory.
I suppose this is the same as #2424 because ./test and test would both reference a file in the current directory, but stripping ./ is terrible IMO. I would prefer that relative paths using . not match anything and #2424 be user error. My reproduce steps are correct, but yours are fine (and more clear) to display this issue too.

It quite the contrary, there's no documentation that a special syntax ./ exists. Although I have to agree that not normalizing paths at all would make more sense.
I guess it makes sense taking a closer look at the exclude path normalization as part of https://github.com/restic/restic/issues/4026 .


Set/Update Iteration on Work Items

Description
As a user, I want to add and remove work items to an/from an iteration by setting the iteration for a work item.
Functional Acceptance Criteria

The user can set the iteration of a work item on the work item detail view. Changes to the iteration are stored on clicking the save button, not in-place.
The user can set the iteration from the work item list. Changes are saved in-place.
The user can select the iteration from a list of open iterations for the current space.

Non-functional Acceptance Criteria
None.
Dependencies

Userstory #409
Userstory #410
Userstory #411

@michaelkleinhenz a user should also be able to directly interact with a work item, in the list, to associate it with an iteration.

@michaelkleinhenz - Are we looking at iterations only from a work item perspective?
The other scenario would be open an iteration and add multiple work items.

@nimishamukherjee currently only from the work item perspective.

@Mgranfie updated the story. I am curious how the UX will look for that :-)

Hi All,
SJ is working on the UI for this, so looping her in to the discussions...
On Fri, Oct 28, 2016 at 3:37 AM, Michael Kleinhenz<EMAIL_ADDRESS>
wrote:

@Mgranfie https://github.com/Mgranfie updated the story. I am curious
how the UX will look for that :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/almighty/almighty-core/issues/412#issuecomment-256856312,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARTwqwQcQS0D_f0D4_1XR7y3vq_rjUlBks5q4aYxgaJpZM4KiheC
.

I think we will want both:

In the work item view, associate the workitem with an iteration.
In an iteration view, assign 1..N workitems to the iteration.

@michaelkleinhenz On # 3 "select the iteration from a list of open iterations for the current project", do you mean that you're on the project page and indicating which iterations are associated with the project?

@Essjaysee no, it means that on either the WI detail view or on the WI list view, the user can select the iteration to be set on the WI from a list of all available iterations. The list of available iterations is scoped to the current project (as every project has an own list of iterations).
Welcome to the Team :-)

I have a question, based on some thinking I was doing for the acceptance criteria. Here are a couple of scenarios:
Given a project PizzaTheHut,
And the project has 3 WIs - a story (first WI) with a UX subtask (2nd) and Coding subtask (3rd),
When the story is assigned to an iteration (sprint X)
Then the subtasks should also be assigned to the iteration.
Given a project PizzaTheHut,
And the project has 3 WIs - a story (first WI) with a UX subtask (2nd) and Coding subtask (3rd),
When the UX subtask is assigned to an iteration (sprint X)
And the coding subtask is assigned to an iteration (sprint X+1)
Then the story would be completed only in sprint X+1.
Would we cover these scenarios as part of this sprint? WIs with relationships may not be that easy to handle in iteration planning, depending on the nature of the WILT.

@michaelkleinhenz Thanks for your reply, makes sense!

@joshuawilson @aslakknutsen @maxandersen @michaelkleinhenz  Wireframes for iterations: https://redhat.invisionapp.com/share/KA9CAYL7M

This issue was moved to fabric8io/fabric8-planner#648


CI: Add initial version II

It seems macOS and ubuntu pass. I think Ubuntu failed in first attempt but passed in the second run here https://github.com/lcompilers/lpython-conda-testing/actions/runs/6291440082/job/17079864280?pr=3.

@certik Is this good to merge?

Only the windows test fails currently. I attempted setting up a virtual widows machine on my Mac, but it didn't succeed. I will attempt this again after the PyCon Event, this time on my linux machine.
For now, we atleast know that the LPython from conda seems to work on Linux and Mac systems.

We'll do best effort o Windows. Linux and macOS are the most important, everything has to work there. Windows we'll iteratively improve


Bug fix for clean resource when shutdown job.

Fixes #702 #686  #683 #605
Changes proposed in this pull request:

Release relative resources when shutdown the job.
Resolve some problems of memory leak.

Coverage decreased (-0.3%) to 82.849% when pulling be375f8c6a762d1354ab1d4a329d22201f77aec0 on dangdang-arch:3.0-bugfix into dd97ea159cb4c0e6df1f41836efb68a0f32cbeb4 on elasticjob:3.0.


Added link

Medical News Today - How to handle impostor syndrome (https://www.medicalnewstoday.com/articles/321730.php)

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @reinvnt
It seems that similar changes have already been accepted earlier for this article you're editing, sorry about that. 😓
If you feel you have more to add, please feel free to open up a new PR.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬


Docs: fixed some grammatical errors

Just fixed some grammatical errors in the docs. And updated the .pre-commit-config.yaml file to avoid mutable reference error in association with a pre-commit issue
Release Note
NONE

Thanks!


Add symlink for src/config directory

Added steps in scaffold.js to add the symlink from drupal docroot/config to src/config.

We do not need to do Drupal core version detection, we should instead make the presence of src/config optional.

Brought this up in a meeting with @grayside. Wanted to link to this issue here: https://www.drupal.org/node/2899534

Basic test coverage is needed. This will require creating examples/src/config/.gitkeep

@grayside or @mike-potter  In Atmosphere we are using config_split. We opted to create our directory structure like this, src/config/default, where default holds our main site configuration. We then have configuration that only exists in specific environments in other directories. Do we think it makes sense to create src/config/default/.gitkeep to make it easier for projects that will need to use config sync? I ask this purely from an organizational standpoint.

There are two basic approaches we can take as far as symlinks: we can link to src/config from build/html/config, or we can link to src/config/* from build/html/config/*. For the latter, I mean the approach taken with symlinking profiles as opposed to the approach of symlinking modules (which uses build/html/modules/custom --> ../../../../src/modules.
I do not think gdt needs to present support for Config Split, but if having a config/default directory seems reasonable in the absence of config split I don't object.

Superceded by #330


Have authMode launch automatically when abortForAuth is called

Currently, when abortForAuth is called, we show an error message.  To match Tableau, we should automatically launch the WDC in auth mode in this scenario.

rolled and ignoring abortForAuth when not in gatherData into PR #99


Evaluation of Prototype Results

Issue Description
The initial prototype with captured traces exists. The next step is to compare the results with a set of ground truth and document the evaluation.
Initializing Ground Truth Set:
For this purpose, a subset of requirements from the Learnify App Repository will be selected and their traces will be documented. This will be a manual analysis made by @codingAku and @ersoykadir.
The selected requirement subset is as follows:

<IP_ADDRESS>.1. Users shall provide their usernames and passwords to log in. - @codingAku
<IP_ADDRESS>.2. Users shall be authenticated after verification and be logged in. - @ersoykadir
<IP_ADDRESS>.1. Users shall be able to annotate post images and texts in learning spaces. @codingAku
<IP_ADDRESS>.2. Users shall be able to view annotations made by other users. - @ersoykadir
<IP_ADDRESS>.1. Users shall be able to see all learning spaces they created or enrolled in. - @codingAku
<IP_ADDRESS>.5.1. Participants shall be able to create community events for that learning space. - @codingAku
<IP_ADDRESS> Users shall be able to edit their profile page. - @ersoykadir
<IP_ADDRESS>.7.1. Participants of a learning space shall be able to create discussion posts. - @codingAku

The results of semantically related traces in the software artifacts will be used as ground truth set for evaluation.
The results of initial prototype:
The tool will be run with two different trace selection methods; keyword extraction and word vectors. The results will be compared with the ground truth set to document recall and precision values.
Step Details
Steps that will be performed:

[x] Create ground truth set
[x] Execute the tool with different trace capture methods
[ ] Document recall and precision values.

Final Actions
The findings need to be documented in a tabular format.
Deadline of the Issue
09.05.2023

The results for keyword-extraction and parent mode:

The results for word-vector and parent-mode:

Recall and precision values documented to paper and submitted.


Cirrus: Remove fixed clone depth

It's been observed on several occasions, some tests fail in git clones
with a "cannot find ref" type error.  Especially in the depth=1 cases.
Since there's really only one place where limiting the depth makes sense
(build-each-commit), simply remove all the other limits.
Signed-off-by: Chris Evich<EMAIL_ADDRESS>

lgtm

/approve

/lgtm