Skip to content

Backport "HBASE-26614 Refactor code related to "dump"ing ZK nodes (#3969)" to branch-2 - #4058

Merged
ndimiduk merged 1 commit into
apache:branch-2from
ndimiduk:26614-refector-zkdump-branch-2
Jan 25, 2022
Merged

Backport "HBASE-26614 Refactor code related to "dump"ing ZK nodes (#3969)" to branch-2#4058
ndimiduk merged 1 commit into
apache:branch-2from
ndimiduk:26614-refector-zkdump-branch-2

Conversation

@ndimiduk

@ndimidukndimiduk commented Jan 24, 2022

Copy link
Copy Markdown
Member

The code starting at ZKUtil.dump(ZKWatcher) is a small mess – it has cyclic dependencies woven
through itself, ZKWatcher and RecoverableZooKeeper. It also initializes a static variable in
ZKUtil through the factory for RecoverableZooKeeper instances. Let's decouple and clean it
up.

Signed-off-by: Duo Zhang zhangduo@apache.org
Signed-off-by: Josh Elser elserj@apache.org

@ndimiduk
HBASE-26614 Refactor code related to "dump"ing ZK nodes (apache#3969)
0da06b0
The code starting at `ZKUtil.dump(ZKWatcher)` is a small mess – it has cyclic dependencies woven
through itself, `ZKWatcher` and `RecoverableZooKeeper`. It also initializes a static variable in
`ZKUtil` through the factory for `RecoverableZooKeeper` instances. Let's decouple and clean it
up.
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Signed-off-by: Josh Elser <elserj@apache.org>
@ndimidukndimiduk added the backport This PR is a back port of some issue or issues already committed to master label Jan 24, 2022
@Apache-HBase

Apache-HBase commented Jan 24, 2022

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec2m 40sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ branch-2 Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for branch
+1 💚mvninstall4m 11sbranch-2 passed
+1 💚compile4m 59sbranch-2 passed
+1 💚checkstyle1m 54sbranch-2 passed
+1 💚spotbugs3m 22sbranch-2 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall3m 53sthe patch passed
+1 💚compile4m 58sthe patch passed
-0 ⚠️javac0m 28shbase-zookeeper generated 5 new + 105 unchanged - 6 fixed = 110 total (was 111)
+1 💚checkstyle1m 52sthe patch passed
+1 💚rubocop0m 9sThere were no new rubocop issues.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck14m 2sPatch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚spotbugs3m 56sthe patch passed
_ Other Tests _
+1 💚asflicense0m 49sThe patch does not generate ASF License warnings.
56m 42s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4058/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4058
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile rubocop
unameLinux 7302fece58a1 4.15.0-162-generic #170-Ubuntu SMP Mon Oct 18 11:38:05 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 5aa0fd2
Default JavaAdoptOpenJDK-1.8.0_282-b08
javachttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4058/1/artifact/yetus-general-check/output/diff-compile-javac-hbase-zookeeper.txt
Max. process+thread count86 (vs. ulimit of 12500)
modulesC: hbase-zookeeper hbase-server hbase-shell hbase-it U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4058/1/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2 rubocop=0.80.0
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Jan 24, 2022

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 35sDocker mode activated.
-0 ⚠️yetus0m 7sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for branch
+1 💚mvninstall4m 43sbranch-2 passed
+1 💚compile2m 37sbranch-2 passed
+1 💚shadedjars8m 27sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 42sbranch-2 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 22sMaven dependency ordering for patch
+1 💚mvninstall4m 29sthe patch passed
+1 💚compile2m 29sthe patch passed
+1 💚javac2m 29sthe patch passed
+1 💚shadedjars7m 37spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 38sthe patch passed
_ Other Tests _
+1 💚unit0m 49shbase-zookeeper in the patch passed.
+1 💚unit136m 15shbase-server in the patch passed.
+1 💚unit7m 11shbase-shell in the patch passed.
+1 💚unit1m 12shbase-it in the patch passed.
183m 32s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4058/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#4058
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 66c70133a83c 4.15.0-156-generic #163-Ubuntu SMP Thu Aug 19 23:31:58 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 5aa0fd2
Default JavaAdoptOpenJDK-11.0.10+9
Test Resultshttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4058/1/testReport/
Max. process+thread count4343 (vs. ulimit of 12500)
modulesC: hbase-zookeeper hbase-server hbase-shell hbase-it U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4058/1/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Jan 24, 2022

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 6sDocker mode activated.
-0 ⚠️yetus0m 6sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for branch
+1 💚mvninstall3m 50sbranch-2 passed
+1 💚compile2m 13sbranch-2 passed
+1 💚shadedjars6m 35sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 27sbranch-2 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for patch
+1 💚mvninstall3m 36sthe patch passed
+1 💚compile2m 15sthe patch passed
+1 💚javac2m 15sthe patch passed
+1 💚shadedjars6m 34spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 29sthe patch passed
_ Other Tests _
+1 💚unit0m 45shbase-zookeeper in the patch passed.
+1 💚unit155m 33shbase-server in the patch passed.
+1 💚unit7m 20shbase-shell in the patch passed.
+1 💚unit1m 17shbase-it in the patch passed.
197m 39s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4058/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#4058
Optional Testsjavac javadoc unit shadedjars compile
unameLinux b8dde8ca488b 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 5aa0fd2
Default JavaAdoptOpenJDK-1.8.0_282-b08
Test Resultshttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4058/1/testReport/
Max. process+thread count3762 (vs. ulimit of 12500)
modulesC: hbase-zookeeper hbase-server hbase-shell hbase-it U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4058/1/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@ndimiduk
ndimiduk merged commit d8085b4 into apache:branch-2Jan 25, 2022
@ndimiduk
ndimiduk deleted the 26614-refector-zkdump-branch-2 branch January 25, 2022 17:08
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.

2 participants

@ndimiduk@Apache-HBase