Skip to content

HBASE-28428 : Zookeeper ConnectionRegistry APIs should have timeout - #6095

Merged
virajjasani merged 4 commits into
apache:branch-2from
Divneet18:HBASE-28428-branch-2
Jul 19, 2024
Merged

HBASE-28428 : Zookeeper ConnectionRegistry APIs should have timeout#6095
virajjasani merged 4 commits into
apache:branch-2from
Divneet18:HBASE-28428-branch-2

Conversation

@Divneet18

Copy link
Copy Markdown
Contributor

No description provided.

@Divneet18Divneet18 reopened this Jul 18, 2024
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 40sDocker mode activated.
-0 ⚠️yetus0m 6sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 9sMaven dependency ordering for branch
+1 💚mvninstall2m 28sbranch-2 passed
+1 💚compile0m 29sbranch-2 passed
+1 💚javadoc0m 26sbranch-2 passed
+1 💚shadedjars5m 11sbranch has no errors when building our shaded downstream artifacts.
-0 ⚠️patch5m 26sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall2m 18sthe patch passed
+1 💚compile0m 32sthe patch passed
+1 💚javac0m 32sthe patch passed
+1 💚javadoc0m 27sthe patch passed
+1 💚shadedjars5m 8spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit7m 56shbase-client in the patch passed.
+1 💚unit0m 40shbase-zookeeper in the patch passed.
28m 3s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#6095
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 8763bd182168 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 toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / ece0bd2
Default JavaTemurin-1.8.0_412-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/testReport/
Max. process+thread count356 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 19sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
_ branch-2 Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for branch
+1 💚mvninstall3m 7sbranch-2 passed
+1 💚compile1m 13sbranch-2 passed
+1 💚checkstyle0m 31sbranch-2 passed
+1 💚spotbugs1m 19sbranch-2 passed
+1 💚spotless0m 47sbranch has no errors when running spotless:check.
-0 ⚠️patch1m 1sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall2m 55sthe patch passed
+1 💚compile1m 11sthe patch passed
+1 💚javac1m 11sthe patch passed
-0 ⚠️blanks0m 0s/blanks-eol.txtThe patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply
+1 💚checkstyle0m 30sthe patch passed
+1 💚spotbugs1m 32sthe patch passed
+1 💚hadoopcheck15m 21sPatch does not cause any errors with Hadoop 2.10.2 or 3.3.6 3.4.0.
-1 ❌spotless0m 22spatch has 22 errors when running spotless:check, run spotless:apply to fix.
_ Other Tests _
+1 💚asflicense0m 22sThe patch does not generate ASF License warnings.
31m 48s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6095
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux 9ecae3d79c2b 5.4.0-182-generic #202-Ubuntu SMP Fri Apr 26 12:29:36 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / ece0bd2
Default JavaEclipse Adoptium-11.0.23+9
spotlesshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/artifact/yetus-general-check/output/patch-spotless.txt
Max. process+thread count77 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/console
versionsgit=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 52sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for branch
+1 💚mvninstall4m 38sbranch-2 passed
+1 💚compile0m 59sbranch-2 passed
+1 💚javadoc0m 48sbranch-2 passed
+1 💚shadedjars6m 36sbranch has no errors when building our shaded downstream artifacts.
-0 ⚠️patch6m 53sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall4m 24sthe patch passed
+1 💚compile0m 48sthe patch passed
+1 💚javac0m 48sthe patch passed
+1 💚javadoc0m 44sthe patch passed
+1 💚shadedjars6m 26spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit7m 37shbase-client in the patch passed.
+1 💚unit0m 57shbase-zookeeper in the patch passed.
36m 49s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6095
Optional Testsjavac javadoc unit compile shadedjars
unameLinux e9cb9ffdaa7a 5.4.0-182-generic #202-Ubuntu SMP Fri Apr 26 12:29:36 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / ece0bd2
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/testReport/
Max. process+thread count377 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 52sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall4m 39sbranch-2 passed
+1 💚compile0m 56sbranch-2 passed
+1 💚javadoc0m 47sbranch-2 passed
+1 💚shadedjars6m 40sbranch has no errors when building our shaded downstream artifacts.
-0 ⚠️patch6m 59sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for patch
+1 💚mvninstall4m 37sthe patch passed
+1 💚compile0m 52sthe patch passed
+1 💚javac0m 52sthe patch passed
+1 💚javadoc0m 41sthe patch passed
+1 💚shadedjars6m 35spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit8m 13shbase-client in the patch passed.
+1 💚unit0m 51shbase-zookeeper in the patch passed.
37m 52s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#6095
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 4ed44fe99bca 5.4.0-182-generic #202-Ubuntu SMP Fri Apr 26 12:29:36 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / ece0bd2
Default JavaEclipse Adoptium-11.0.23+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/testReport/
Max. process+thread count387 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/1/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 44sDocker mode activated.
-0 ⚠️yetus0m 6sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 20sMaven dependency ordering for branch
+1 💚mvninstall2m 16sbranch-2 passed
+1 💚compile0m 30sbranch-2 passed
+1 💚javadoc0m 25sbranch-2 passed
+1 💚shadedjars5m 7sbranch has no errors when building our shaded downstream artifacts.
-0 ⚠️patch5m 21sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 20sthe patch passed
+1 💚compile0m 32sthe patch passed
+1 💚javac0m 32sthe patch passed
+1 💚javadoc0m 27sthe patch passed
+1 💚shadedjars5m 9spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit8m 4shbase-client in the patch passed.
+1 💚unit0m 39shbase-zookeeper in the patch passed.
28m 1s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#6095
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 9c0b9a3e3035 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 toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 6e930a8
Default JavaTemurin-1.8.0_412-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/testReport/
Max. process+thread count344 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 20sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
_ branch-2 Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for branch
+1 💚mvninstall2m 56sbranch-2 passed
+1 💚compile1m 12sbranch-2 passed
+1 💚checkstyle0m 29sbranch-2 passed
+1 💚spotbugs1m 15sbranch-2 passed
+1 💚spotless0m 45sbranch has no errors when running spotless:check.
-0 ⚠️patch1m 0sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall2m 55sthe patch passed
+1 💚compile1m 12sthe patch passed
+1 💚javac1m 12sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 29sthe patch passed
+1 💚spotbugs1m 31sthe patch passed
+1 💚hadoopcheck15m 4sPatch does not cause any errors with Hadoop 2.10.2 or 3.3.6 3.4.0.
+1 💚spotless0m 45spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 21sThe patch does not generate ASF License warnings.
31m 23s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6095
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux e1befd0b20d6 5.4.0-182-generic #202-Ubuntu SMP Fri Apr 26 12:29:36 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 6e930a8
Default JavaEclipse Adoptium-11.0.23+9
Max. process+thread count79 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/console
versionsgit=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 46sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall4m 32sbranch-2 passed
+1 💚compile0m 51sbranch-2 passed
+1 💚javadoc0m 41sbranch-2 passed
+1 💚shadedjars6m 31sbranch has no errors when building our shaded downstream artifacts.
-0 ⚠️patch6m 47sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall4m 27sthe patch passed
+1 💚compile0m 47sthe patch passed
+1 💚javac0m 47sthe patch passed
+1 💚javadoc0m 41sthe patch passed
+1 💚shadedjars6m 26spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit7m 37shbase-client in the patch passed.
+1 💚unit0m 56shbase-zookeeper in the patch passed.
36m 2s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6095
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 419d050562ef 5.4.0-182-generic #202-Ubuntu SMP Fri Apr 26 12:29:36 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 6e930a8
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/testReport/
Max. process+thread count379 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 53sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall4m 42sbranch-2 passed
+1 💚compile0m 50sbranch-2 passed
+1 💚javadoc0m 42sbranch-2 passed
+1 💚shadedjars6m 37sbranch has no errors when building our shaded downstream artifacts.
-0 ⚠️patch6m 57sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗mvndep0m 21sMaven dependency ordering for patch
+1 💚mvninstall4m 35sthe patch passed
+1 💚compile0m 49sthe patch passed
+1 💚javac0m 49sthe patch passed
+1 💚javadoc0m 40sthe patch passed
+1 💚shadedjars6m 28spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit8m 4shbase-client in the patch passed.
+1 💚unit0m 50shbase-zookeeper in the patch passed.
37m 0s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#6095
Optional Testsjavac javadoc unit compile shadedjars
unameLinux c6992a83604c 5.4.0-182-generic #202-Ubuntu SMP Fri Apr 26 12:29:36 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 6e930a8
Default JavaEclipse Adoptium-11.0.23+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/testReport/
Max. process+thread count382 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6095/2/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@virajjasanivirajjasani left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@virajjasani
virajjasani merged commit 1d8bc46 into apache:branch-2Jul 19, 2024
virajjasani pushed a commit that referenced this pull request Jul 19, 2024
…6095) (#5837)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Signed-off-by: Pankaj Kumar <pankajkumar@apache.org>
Signed-off-by: Viraj Jasani <vjasani@apache.org>
virajjasani pushed a commit that referenced this pull request Jul 19, 2024
…6095) (#5837)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Signed-off-by: Pankaj Kumar <pankajkumar@apache.org>
Signed-off-by: Viraj Jasani <vjasani@apache.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Divneet18@Apache-HBase@virajjasani