Skip to content

HBASE-28745 : Default Zookeeper ConnectionRegistry APIs timeout should be less - #6104

Merged
virajjasani merged 3 commits into
apache:branch-2from
Divneet18:HBASE-28745-branch-2
Jul 20, 2024
Merged

HBASE-28745 : Default Zookeeper ConnectionRegistry APIs timeout should be less#6104
virajjasani merged 3 commits into
apache:branch-2from
Divneet18:HBASE-28745-branch-2

Conversation

@Divneet18

@Divneet18Divneet18 commented Jul 19, 2024

Copy link
Copy Markdown
Contributor

Addendum for HBASE-28428

@virajjasani

virajjasani commented Jul 19, 2024

Copy link
Copy Markdown
Contributor

Let's keep it 10s default?

@Apache-HBase

Apache-HBase commented Jul 19, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 45sDocker 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 💚mvninstall3m 6sbranch-2 passed
+1 💚compile1m 8sbranch-2 passed
+1 💚checkstyle0m 25sbranch-2 passed
+1 💚spotbugs1m 11sbranch-2 passed
+1 💚spotless0m 46sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
+1 💚mvninstall2m 53sthe patch passed
+1 💚compile1m 6sthe patch passed
+1 💚javac1m 6sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 25sthe patch passed
+1 💚spotbugs1m 24sthe patch passed
+1 💚hadoopcheck16m 0sPatch 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 18sThe patch does not generate ASF License warnings.
32m 41s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6104
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux 46c8f4f66805 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 / 241bf81
Default JavaEclipse Adoptium-11.0.23+9
Max. process+thread count76 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/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

Apache-HBase commented Jul 19, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 27sDocker 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 14sMaven dependency ordering for branch
+1 💚mvninstall3m 56sbranch-2 passed
+1 💚compile0m 43sbranch-2 passed
+1 💚javadoc0m 46sbranch-2 passed
+1 💚shadedjars6m 18sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for patch
+1 💚mvninstall2m 58sthe patch passed
+1 💚compile0m 40sthe patch passed
+1 💚javac0m 40sthe patch passed
+1 💚javadoc0m 34sthe patch passed
+1 💚shadedjars5m 31spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit8m 12shbase-client in the patch passed.
+1 💚unit0m 43shbase-zookeeper in the patch passed.
32m 52s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#6104
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 0addb5f3ce43 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 / 241bf81
Default JavaEclipse Adoptium-11.0.23+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/1/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-6104/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

Apache-HBase commented Jul 19, 2024

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 52sDocker 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 10sMaven dependency ordering for branch
-1 ❌mvninstall2m 58s/branch-mvninstall-root.txtroot in branch-2 failed.
+1 💚compile0m 39sbranch-2 passed
+1 💚javadoc0m 34sbranch-2 passed
+1 💚shadedjars7m 21sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
+1 💚mvninstall3m 17sthe patch passed
+1 💚compile0m 39sthe patch passed
+1 💚javac0m 39sthe patch passed
+1 💚javadoc0m 37sthe patch passed
+1 💚shadedjars6m 23spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit8m 16shbase-client in the patch passed.
+1 💚unit0m 46shbase-zookeeper in the patch passed.
34m 19s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#6104
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 52c818fd1705 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 / 241bf81
Default JavaTemurin-1.8.0_412-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/1/testReport/
Max. process+thread count359 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/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

Apache-HBase commented Jul 19, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec1m 10sDocker mode activated.
-0 ⚠️yetus0m 4sUnprocessed 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 52sbranch-2 passed
+1 💚javadoc0m 42sbranch-2 passed
+1 💚shadedjars7m 12sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
+1 💚mvninstall4m 7sthe patch passed
+1 💚compile0m 51sthe patch passed
+1 💚javac0m 51sthe patch passed
+1 💚javadoc0m 42sthe patch passed
+1 💚shadedjars6m 14spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit8m 29shbase-client in the patch passed.
+1 💚unit0m 50shbase-zookeeper in the patch passed.
37m 45s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6104
Optional Testsjavac javadoc unit compile shadedjars
unameLinux adeadc5949b4 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 241bf81
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/1/testReport/
Max. process+thread count375 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/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.

@virajjasani

virajjasani commented Jul 19, 2024

Copy link
Copy Markdown
Contributor

@Divneet18 could you create PR against master branch?

@virajjasanivirajjasani added the backport This PR is a back port of some issue or issues already committed to master label Jul 20, 2024
@Apache-HBase

Apache-HBase commented Jul 20, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 19sDocker 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 9sMaven dependency ordering for branch
+1 💚mvninstall3m 1sbranch-2 passed
+1 💚compile0m 40sbranch-2 passed
+1 💚javadoc0m 33sbranch-2 passed
+1 💚shadedjars5m 35sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for patch
+1 💚mvninstall2m 57sthe patch passed
+1 💚compile0m 40sthe patch passed
+1 💚javac0m 40sthe patch passed
+1 💚javadoc0m 34sthe patch passed
+1 💚shadedjars5m 30spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit8m 5shbase-client in the patch passed.
+1 💚unit0m 44shbase-zookeeper in the patch passed.
30m 33s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#6104
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 0896ebf470f9 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 / 241bf81
Default JavaEclipse Adoptium-11.0.23+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/2/testReport/
Max. process+thread count381 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/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

Apache-HBase commented Jul 20, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 43sDocker 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 💚mvninstall3m 7sbranch-2 passed
+1 💚compile0m 44sbranch-2 passed
+1 💚javadoc0m 34sbranch-2 passed
+1 💚shadedjars5m 36sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall3m 3sthe patch passed
+1 💚compile0m 41sthe patch passed
+1 💚javac0m 41sthe patch passed
+1 💚javadoc0m 34sthe patch passed
+1 💚shadedjars5m 28spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit8m 14shbase-client in the patch passed.
+1 💚unit0m 47shbase-zookeeper in the patch passed.
31m 27s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6104
Optional Testsjavac javadoc unit compile shadedjars
unameLinux fa2399f2ed31 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 241bf81
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/2/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-6104/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

Apache-HBase commented Jul 20, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 43sDocker 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 20sMaven dependency ordering for branch
+1 💚mvninstall3m 0sbranch-2 passed
+1 💚compile1m 7sbranch-2 passed
+1 💚checkstyle0m 24sbranch-2 passed
+1 💚spotbugs1m 8sbranch-2 passed
+1 💚spotless0m 43sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 50sthe patch passed
+1 💚compile1m 6sthe patch passed
+1 💚javac1m 6sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 24sthe patch passed
+1 💚spotbugs1m 25sthe patch passed
+1 💚hadoopcheck15m 39sPatch does not cause any errors with Hadoop 2.10.2 or 3.3.6 3.4.0.
+1 💚spotless0m 44spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 16sThe patch does not generate ASF License warnings.
32m 0s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6104
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux 35d9e2b7870c 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 / 241bf81
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-6104/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

Apache-HBase commented Jul 20, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 48sDocker 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 57sbranch-2 passed
+1 💚compile0m 39sbranch-2 passed
+1 💚javadoc0m 31sbranch-2 passed
+1 💚shadedjars6m 39sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall3m 10sthe patch passed
+1 💚compile0m 42sthe patch passed
+1 💚javac0m 42sthe patch passed
+1 💚javadoc0m 36sthe patch passed
+1 💚shadedjars6m 5spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit8m 17shbase-client in the patch passed.
+1 💚unit0m 48shbase-zookeeper in the patch passed.
33m 22s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#6104
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 004ed6bf4ca7 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 / 241bf81
Default JavaTemurin-1.8.0_412-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/2/testReport/
Max. process+thread count358 (vs. ulimit of 30000)
modulesC: hbase-client hbase-zookeeper U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6104/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.

@virajjasani
virajjasani merged commit bd00e69 into apache:branch-2Jul 20, 2024
virajjasani pushed a commit that referenced this pull request Jul 20, 2024
@Divneet18@virajjasani
HBASE-28745: Default Zookeeper ConnectionRegistry APIs timeout shoul…
3b4c214
…d be less (#6104) (#6105)
Signed-off-by: Viraj Jasani <vjasani@apache.org>
virajjasani pushed a commit that referenced this pull request Jul 20, 2024
@Divneet18@virajjasani
HBASE-28745: Default Zookeeper ConnectionRegistry APIs timeout shoul…
e3c513b
…d be less (#6104) (#6105)
Signed-off-by: Viraj Jasani <vjasani@apache.org>
sanjeet006py pushed a commit to sanjeet006py/hbase that referenced this pull request Aug 24, 2025
@Divneet18@mnpoonia
HBASE-28745 : Default Zookeeper ConnectionRegistry APIs timeout shoul…
5a8c2d7
…d be less (apache#6104) (apache#6105)
Signed-off-by: Viraj Jasani <vjasani@apache.org>
Signed-off-by: apoonia <apoonia@salesforce.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

backportThis PR is a back port of some issue or issues already committed to master

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Divneet18@virajjasani@Apache-HBase