HBASE-25853 Backport HBASE-22120 "Replace HTrace with OpenTelemetry" to branch-2 by taklwu · Pull Request #3637 · apache/hbase

taklwu · 2021-08-27T16:46:44Z

dropped the commits (1a30692 and a03ce5a) of HBASE-26176 in feature HBASE-25853 for running jenkins build for hadoop3
rebased feature HBASE-25853 onto branch-2, focus pushed to HBASE-25853 and it's a clean rebase

…se and try to make use of OpenTelemetry" to branch-2 (#3529) 1/17 commits of HBASE-22120 Signed-off-by: Peter Somogyi <psomogyi@apache.org>

… rpc client" to branch-2 (#3543) 2/17 commits of HBASE-22120 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

…tracing without directly depending on opentelemetry-sdk" to branch-2 (#3552) 3/17 commits of HBASE-22120, original commit#57960fa8fa7228d65b1a4adc8e9b5b1a8158824d Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

#3556) 4/17 commits of HBASE-22120, original commit 805b2ae Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

#3561) 5/17 commits of HBASE-22120, original commit dcb78bd and merged conflicts after rebasing on HBASE-26150 with commit 63d4970 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

…acing rpc call at client side" to branch-2 (#3589) 6/17 commits of HBASE-22120, original commit ae2c62f Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

#3594) 7/17 commits of HBASE-22120, original commit 03e12bf Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

…branch-2 (#3597) 8/17 commits of HBASE-22120, original commit 2be2c63 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

…RpcClient" to branch-2 (#3607) 9/17 commits of HBASE-22120, original commit bb8c496 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

… branch-2 (#3608) 10/17 commits of HBASE-22120, original commit f6ff519 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

…#3618) 11/17 commits of HBASE-22120, original commit 8d68f8c Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

…branch-2 (#3620) 12/17 commits of HBASE-22120, original commits 8399293 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

…t for RegionScanner.next" to branch-2 (#3623) 13/17 commits of HBASE-22120, original commit 7f90c22 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

…r tracing after upgrading opentelemtry to 1.0.0" to branch-2 (#3624) 14/17 commits of HBASE-22120, original commit 8df9beb Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

…branch-2 (#3628) 15/17 commits of HBASE-22120, original commit b714889 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

…and view tracing with OpenTelemetry" to branch-2 (#3629) 16/17 commits of HBASE-22120, original commit be4503d Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

…ncConnectionImpl.getHbck is incorrect" to branch-2 (#3631) 17/17 commits of HBASE-22120, original commit f36e153 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

…ing after HBASE-22120 merged" into branch-2 (#3634) move opentelemetry jars to client-facing-thirdparty add opentelemetry jars when init map reduce job dependencies original commit 8d2a0ef Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

Apache-HBase · 2021-08-27T18:43:51Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	1m 7s	Docker mode activated.
		_ Prechecks _
+1 💚	dupname	0m 1s	No case conflicting files found.
+0 🆗	shelldocs	0m 1s	Shelldocs was not available.
+0 🆗	prototool	0m 0s	prototool was not available.
+1 💚	hbaseanti	0m 0s	Patch does not have any anti-patterns.
+1 💚	@author	0m 0s	The patch does not contain any @author tags.
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 25s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 35s	branch-2 passed
+1 💚	compile	8m 47s	branch-2 passed
+1 💚	checkstyle	2m 23s	branch-2 passed
+0 🆗	refguide	3m 45s	branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
+1 💚	spotbugs	17m 46s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 24s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 17s	the patch passed
+1 💚	compile	8m 45s	the patch passed
+1 💚	cc	8m 45s	the patch passed
+1 💚	javac	8m 45s	the patch passed
+1 💚	checkstyle	2m 23s	root: The patch generated 0 new + 800 unchanged - 33 fixed = 800 total (was 833)
+1 💚	rubocop	0m 5s	The patch generated 0 new + 3 unchanged - 8 fixed = 3 total (was 11)
+1 💚	shellcheck	0m 2s	There were no new shellcheck issues.
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	xml	0m 19s	The patch has no ill-formed XML file.
+0 🆗	refguide	3m 37s	patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
+1 💚	hadoopcheck	12m 10s	Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚	hbaseprotoc	12m 52s	the patch passed
+1 💚	spotbugs	19m 46s	the patch passed
		_ Other Tests _
+1 💚	asflicense	3m 12s	The patch does not generate ASF License warnings.
		116m 9s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	dupname asflicense shellcheck shelldocs javac hadoopcheck xml compile spotbugs hbaseanti checkstyle cc hbaseprotoc prototool rubocop refguide
uname	Linux 425e699d2eb9 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `30bb72b`
Default Java	AdoptOpenJDK-1.8.0_282-b08
refguide	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/artifact/yetus-general-check/output/branch-site/book.html
refguide	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/artifact/yetus-general-check/output/patch-site/book.html
Max. process+thread count	141 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/console
versions	git=2.17.1 maven=3.6.3 shellcheck=0.4.6 spotbugs=4.2.2 rubocop=0.80.0
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-27T20:14:42Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 33s	Docker mode activated.
-0 ⚠️	yetus	0m 7s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 26s	Maven dependency ordering for branch
+1 💚	mvninstall	4m 15s	branch-2 passed
+1 💚	compile	2m 49s	branch-2 passed
+1 💚	shadedjars	6m 56s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 58s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 23s	Maven dependency ordering for patch
+1 💚	mvninstall	4m 2s	the patch passed
+1 💚	compile	2m 46s	the patch passed
+1 💚	javac	2m 46s	the patch passed
+1 💚	shadedjars	7m 17s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 57s	the patch passed
		_ Other Tests _
-1 ❌	unit	159m 7s	root in the patch failed.
		207m 6s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux be0d3c5f3c2e 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `30bb72b`
Default Java	AdoptOpenJDK-11.0.10+9
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/testReport/
Max. process+thread count	3532 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/console
versions	git=2.17.1 maven=3.6.3
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-27T22:53:50Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 37s	Docker mode activated.
-0 ⚠️	yetus	0m 5s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 28s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 54s	branch-2 passed
+1 💚	compile	2m 24s	branch-2 passed
+1 💚	shadedjars	6m 22s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 32s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 25s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 37s	the patch passed
+1 💚	compile	2m 27s	the patch passed
+1 💚	javac	2m 27s	the patch passed
+1 💚	shadedjars	6m 47s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 36s	the patch passed
		_ Other Tests _
-1 ❌	unit	320m 39s	root in the patch failed.
		365m 42s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux 4fe04878f62a 4.15.0-151-generic #157-Ubuntu SMP Fri Jul 9 23:07:57 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `30bb72b`
Default Java	AdoptOpenJDK-1.8.0_282-b08
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/testReport/
Max. process+thread count	4075 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/1/console
versions	git=2.17.1 maven=3.6.3
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

taklwu · 2021-08-28T18:31:19Z

the failed test TestInfoServersACL is related to HBASE-24330, and it's not related to our changes here. (rerunning to see if it could pass in the second run)

Apache-HBase · 2021-08-28T20:42:23Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 34s	Docker mode activated.
		_ Prechecks _
+1 💚	dupname	0m 1s	No case conflicting files found.
+0 🆗	shelldocs	0m 1s	Shelldocs was not available.
+0 🆗	prototool	0m 0s	prototool was not available.
+1 💚	hbaseanti	0m 0s	Patch does not have any anti-patterns.
+1 💚	@author	0m 0s	The patch does not contain any @author tags.
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 29s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 55s	branch-2 passed
+1 💚	compile	9m 54s	branch-2 passed
+1 💚	checkstyle	2m 29s	branch-2 passed
+0 🆗	refguide	3m 59s	branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
+1 💚	spotbugs	20m 18s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 25s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 34s	the patch passed
+1 💚	compile	10m 27s	the patch passed
+1 💚	cc	10m 27s	the patch passed
+1 💚	javac	10m 27s	the patch passed
+1 💚	checkstyle	2m 47s	root: The patch generated 0 new + 800 unchanged - 33 fixed = 800 total (was 833)
+1 💚	rubocop	0m 5s	The patch generated 0 new + 3 unchanged - 8 fixed = 3 total (was 11)
+1 💚	shellcheck	0m 2s	There were no new shellcheck issues.
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	xml	0m 24s	The patch has no ill-formed XML file.
+0 🆗	refguide	5m 16s	patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
+1 💚	hadoopcheck	13m 12s	Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚	hbaseprotoc	13m 45s	the patch passed
+1 💚	spotbugs	22m 36s	the patch passed
		_ Other Tests _
+1 💚	asflicense	3m 21s	The patch does not generate ASF License warnings.
		129m 48s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	dupname asflicense shellcheck shelldocs javac hadoopcheck xml compile spotbugs hbaseanti checkstyle cc hbaseprotoc prototool rubocop refguide
uname	Linux 8a47ee2f7b1a 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `033ff75`
Default Java	AdoptOpenJDK-1.8.0_282-b08
refguide	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/artifact/yetus-general-check/output/branch-site/book.html
refguide	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/artifact/yetus-general-check/output/patch-site/book.html
Max. process+thread count	141 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/console
versions	git=2.17.1 maven=3.6.3 shellcheck=0.4.6 spotbugs=4.2.2 rubocop=0.80.0
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-28T22:12:44Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 32s	Docker mode activated.
-0 ⚠️	yetus	0m 7s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 27s	Maven dependency ordering for branch
+1 💚	mvninstall	4m 18s	branch-2 passed
+1 💚	compile	2m 49s	branch-2 passed
+1 💚	shadedjars	6m 53s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 59s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 24s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 55s	the patch passed
+1 💚	compile	2m 51s	the patch passed
+1 💚	javac	2m 51s	the patch passed
+1 💚	shadedjars	7m 14s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 57s	the patch passed
		_ Other Tests _
+1 💚	unit	171m 11s	root in the patch passed.
		220m 24s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux e7cff0c8acc5 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `033ff75`
Default Java	AdoptOpenJDK-11.0.10+9
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/testReport/
Max. process+thread count	5848 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/console
versions	git=2.17.1 maven=3.6.3
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-29T00:38:09Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	2m 40s	Docker mode activated.
-0 ⚠️	yetus	0m 6s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 25s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 35s	branch-2 passed
+1 💚	compile	2m 15s	branch-2 passed
+1 💚	shadedjars	6m 2s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 0s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 24s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 20s	the patch passed
+1 💚	compile	2m 15s	the patch passed
+1 💚	javac	2m 15s	the patch passed
+1 💚	shadedjars	6m 28s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 5s	the patch passed
		_ Other Tests _
-1 ❌	unit	321m 28s	root in the patch failed.
		365m 46s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux 0f3eb70d04f1 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `033ff75`
Default Java	AdoptOpenJDK-1.8.0_282-b08
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/testReport/
Max. process+thread count	4068 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/2/console
versions	git=2.17.1 maven=3.6.3
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache9 · 2021-08-29T04:56:16Z

Oh my reply is missing? Or maybe I just forgot to hit the comment button to send it out...

There is a difference between master and branch-2 is that branch-2 still have a separated sync client implementation which is not built upon the async client, so we should also implement tracing for the sync client.

I think I have raised this point before but I can not recall what is final plan... So we want to backport all the commits from master to branch-2 first, and then implement tracing for sync client on branch-2? Just want to confirm the plan, I do not mean you must implement tracing for sync client in this PR.

Thanks.

taklwu · 2021-08-30T02:54:59Z

we want to backport all the commits from master to branch-2 first, and then implement tracing for sync client on branch-2? Just want to confirm the plan, I do not mean you must implement tracing for sync client in this PR.

@Apache9 right, I saw that comment about sync client and created a followup tasks/story HBASE-26141 for implementing trace support for sync client.

and yeah, these set of commits for HBASE-25853 just first backport everything.

Apache9 · 2021-08-30T02:58:10Z

OK, got it.

Let's open a VOTE thread in the dev list. The merge back of a feature branch should get at least 3 +1s from committers.

No worry, you already have mine :)

Thanks.

taklwu · 2021-08-30T03:00:13Z

will do ! and thanks for your help on reviewing all those commits.

taklwu · 2021-08-30T03:25:18Z

yetus hadoop3 unit tests passed in second run, and TestInfoServersACL didn't showup.

rerunning another time for yetus hadoop2 unit tests but the test report of second run did not find anything related to this change.

Apache-HBase · 2021-08-30T05:35:03Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 33s	Docker mode activated.
		_ Prechecks _
+1 💚	dupname	0m 1s	No case conflicting files found.
+0 🆗	shelldocs	0m 1s	Shelldocs was not available.
+0 🆗	prototool	0m 1s	prototool was not available.
+1 💚	hbaseanti	0m 0s	Patch does not have any anti-patterns.
+1 💚	@author	0m 0s	The patch does not contain any @author tags.
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 30s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 59s	branch-2 passed
+1 💚	compile	10m 30s	branch-2 passed
+1 💚	checkstyle	2m 38s	branch-2 passed
+0 🆗	refguide	4m 6s	branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
+1 💚	spotbugs	21m 48s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 27s	Maven dependency ordering for patch
+1 💚	mvninstall	4m 22s	the patch passed
+1 💚	compile	13m 5s	the patch passed
+1 💚	cc	13m 5s	the patch passed
+1 💚	javac	13m 5s	the patch passed
+1 💚	checkstyle	3m 29s	root: The patch generated 0 new + 800 unchanged - 33 fixed = 800 total (was 833)
+1 💚	rubocop	0m 6s	The patch generated 0 new + 3 unchanged - 8 fixed = 3 total (was 11)
+1 💚	shellcheck	0m 2s	There were no new shellcheck issues.
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	xml	0m 27s	The patch has no ill-formed XML file.
+0 🆗	refguide	6m 6s	patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
+1 💚	hadoopcheck	17m 38s	Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚	hbaseprotoc	18m 41s	the patch passed
+1 💚	spotbugs	28m 15s	the patch passed
		_ Other Tests _
+1 💚	asflicense	3m 21s	The patch does not generate ASF License warnings.
		154m 48s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	dupname asflicense shellcheck shelldocs javac hadoopcheck xml compile spotbugs hbaseanti checkstyle cc hbaseprotoc prototool rubocop refguide
uname	Linux b8adae5895cd 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `1e49e94`
Default Java	AdoptOpenJDK-1.8.0_282-b08
refguide	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/artifact/yetus-general-check/output/branch-site/book.html
refguide	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/artifact/yetus-general-check/output/patch-site/book.html
Max. process+thread count	142 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/console
versions	git=2.17.1 maven=3.6.3 shellcheck=0.4.6 spotbugs=4.2.2 rubocop=0.80.0
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-30T06:15:39Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 33s	Docker mode activated.
-0 ⚠️	yetus	0m 6s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 36s	Maven dependency ordering for branch
+1 💚	mvninstall	4m 3s	branch-2 passed
+1 💚	compile	2m 46s	branch-2 passed
+1 💚	shadedjars	6m 49s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 56s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 20s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 58s	the patch passed
+1 💚	compile	2m 50s	the patch passed
+1 💚	javac	2m 50s	the patch passed
+1 💚	shadedjars	7m 15s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	7m 1s	the patch passed
		_ Other Tests _
-1 ❌	unit	147m 46s	root in the patch failed.
		195m 45s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux b512a3a17c0f 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `1e49e94`
Default Java	AdoptOpenJDK-11.0.10+9
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/testReport/
Max. process+thread count	3451 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/console
versions	git=2.17.1 maven=3.6.3
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-30T09:23:22Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 30s	Docker mode activated.
-0 ⚠️	yetus	0m 7s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 36s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 34s	branch-2 passed
+1 💚	compile	2m 30s	branch-2 passed
+1 💚	shadedjars	6m 25s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 39s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 21s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 37s	the patch passed
+1 💚	compile	2m 28s	the patch passed
+1 💚	javac	2m 28s	the patch passed
+1 💚	shadedjars	6m 47s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 33s	the patch passed
		_ Other Tests _
-1 ❌	unit	337m 22s	root in the patch failed.
		382m 13s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux b05ae2017cc4 4.15.0-151-generic #157-Ubuntu SMP Fri Jul 9 23:07:57 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `1e49e94`
Default Java	AdoptOpenJDK-1.8.0_282-b08
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/testReport/
Max. process+thread count	4680 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/3/console
versions	git=2.17.1 maven=3.6.3
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

taklwu · 2021-08-30T16:39:55Z

since hadoop3 unit tests passed once , I only look at the latest failure in hadoop2, those unrelated tests are TestClusterScopeQuotaThrottle,TestReplicationEmptyWALRecovery in hbase-server , TestSecureExportSnapshot,TestMobSecureExportSnapshot in hbase-mapreducer. I confirmed they're working by running them manually and please see the result below.

Also, I got 3 +1s from the vote thread and will wait for another day before merging.

cd hbase-server
mvn clean test -Dtest=TestClusterScopeQuotaThrottle,TestReplicationEmptyWALRecovery
[INFO] -------------------------------------------------------
[INFO] T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.hadoop.hbase.replication.TestReplicationEmptyWALRecovery
[INFO] Running org.apache.hadoop.hbase.quotas.TestClusterScopeQuotaThrottle
[INFO] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 48.372 s - in org.apache.hadoop.hbase.replication.TestReplicationEmptyWALRecovery
[WARNING] Tests run: 5, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 111.231 s - in org.apache.hadoop.hbase.quotas.TestClusterScopeQuotaThrottle
[INFO]
[INFO] Results:
[INFO]
[WARNING] Tests run: 9, Failures: 0, Errors: 0, Skipped: 1
[INFO]
[INFO]
[INFO] --- maven-surefire-plugin:3.0.0-M4:test (secondPartTestsExecution) @ hbase-server ---
[INFO] Tests are skipped.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 03:36 min
[INFO] Finished at: 2021-08-30T09:29:03-07:00
[INFO] ------------------------------------------------------------------------
cd ../hbase-mapreduce
mvn clean test -Dtest=TestSecureExportSnapshot,TestMobSecureExportSnapshot
[INFO] -------------------------------------------------------
[INFO] T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.hadoop.hbase.snapshot.TestSecureExportSnapshot
[INFO] Running org.apache.hadoop.hbase.snapshot.TestMobSecureExportSnapshot
[INFO] Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 190.275 s - in org.apache.hadoop.hbase.snapshot.TestMobSecureExportSnapshot
[INFO] Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 190.764 s - in org.apache.hadoop.hbase.snapshot.TestSecureExportSnapshot
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 10, Failures: 0, Errors: 0, Skipped: 0
[INFO]
[INFO]
[INFO] --- maven-surefire-plugin:3.0.0-M4:test (secondPartTestsExecution) @ hbase-mapreduce ---
[INFO] Tests are skipped.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 03:29 min
[INFO] Finished at: 2021-08-30T09:36:07-07:00
[INFO] ------------------------------------------------------------------------

Apache-HBase · 2021-08-31T02:30:57Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 34s	Docker mode activated.
		_ Prechecks _
+1 💚	dupname	0m 1s	No case conflicting files found.
+0 🆗	shelldocs	0m 1s	Shelldocs was not available.
+0 🆗	prototool	0m 0s	prototool was not available.
+1 💚	hbaseanti	0m 0s	Patch does not have any anti-patterns.
+1 💚	@author	0m 0s	The patch does not contain any @author tags.
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 42s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 36s	branch-2 passed
+1 💚	compile	9m 48s	branch-2 passed
+1 💚	checkstyle	2m 28s	branch-2 passed
+0 🆗	refguide	3m 37s	branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
+1 💚	spotbugs	20m 2s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 20s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 36s	the patch passed
+1 💚	compile	9m 56s	the patch passed
+1 💚	cc	9m 56s	the patch passed
+1 💚	javac	9m 56s	the patch passed
+1 💚	checkstyle	2m 29s	root: The patch generated 0 new + 800 unchanged - 33 fixed = 800 total (was 833)
+1 💚	rubocop	0m 4s	The patch generated 0 new + 3 unchanged - 8 fixed = 3 total (was 11)
+1 💚	shellcheck	0m 1s	There were no new shellcheck issues.
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	xml	0m 22s	The patch has no ill-formed XML file.
+0 🆗	refguide	3m 41s	patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
+1 💚	hadoopcheck	12m 24s	Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚	hbaseprotoc	13m 33s	the patch passed
+1 💚	spotbugs	22m 20s	the patch passed
		_ Other Tests _
+1 💚	asflicense	3m 6s	The patch does not generate ASF License warnings.
		124m 48s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	dupname asflicense shellcheck shelldocs javac hadoopcheck xml compile spotbugs hbaseanti checkstyle cc hbaseprotoc prototool rubocop refguide
uname	Linux bb0b5ebd0069 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `c08e877`
Default Java	AdoptOpenJDK-1.8.0_282-b08
refguide	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/artifact/yetus-general-check/output/branch-site/book.html
refguide	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/artifact/yetus-general-check/output/patch-site/book.html
Max. process+thread count	141 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/console
versions	git=2.17.1 maven=3.6.3 shellcheck=0.4.6 spotbugs=4.2.2 rubocop=0.80.0
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-31T03:38:33Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 32s	Docker mode activated.
-0 ⚠️	yetus	0m 8s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 37s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 54s	branch-2 passed
+1 💚	compile	2m 49s	branch-2 passed
+1 💚	shadedjars	6m 51s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 55s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 20s	Maven dependency ordering for patch
+1 💚	mvninstall	4m 2s	the patch passed
+1 💚	compile	2m 51s	the patch passed
+1 💚	javac	2m 51s	the patch passed
+1 💚	shadedjars	7m 12s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	7m 1s	the patch passed
		_ Other Tests _
-1 ❌	unit	144m 45s	root in the patch failed.
		192m 49s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux acea0e04e923 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `c08e877`
Default Java	AdoptOpenJDK-11.0.10+9
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/testReport/
Max. process+thread count	4043 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/console
versions	git=2.17.1 maven=3.6.3
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-31T07:03:27Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 33s	Docker mode activated.
-0 ⚠️	yetus	0m 5s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 39s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 46s	branch-2 passed
+1 💚	compile	2m 34s	branch-2 passed
+1 💚	shadedjars	6m 42s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	7m 5s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 21s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 53s	the patch passed
+1 💚	compile	2m 41s	the patch passed
+1 💚	javac	2m 41s	the patch passed
+1 💚	shadedjars	7m 14s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	7m 8s	the patch passed
		_ Other Tests _
-1 ❌	unit	348m 50s	root in the patch failed.
		396m 24s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux ae2ad20b24b0 4.15.0-151-generic #157-Ubuntu SMP Fri Jul 9 23:07:57 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `c08e877`
Default Java	AdoptOpenJDK-1.8.0_282-b08
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/testReport/
Max. process+thread count	4289 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/4/console
versions	git=2.17.1 maven=3.6.3
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-31T18:54:07Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 43s	Docker mode activated.
		_ Prechecks _
+1 💚	dupname	0m 1s	No case conflicting files found.
+0 🆗	shelldocs	0m 1s	Shelldocs was not available.
+0 🆗	prototool	0m 0s	prototool was not available.
+1 💚	hbaseanti	0m 0s	Patch does not have any anti-patterns.
+1 💚	@author	0m 0s	The patch does not contain any @author tags.
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 44s	Maven dependency ordering for branch
+1 💚	mvninstall	5m 24s	branch-2 passed
+1 💚	compile	13m 48s	branch-2 passed
+1 💚	checkstyle	3m 29s	branch-2 passed
+0 🆗	refguide	4m 45s	branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
+1 💚	spotbugs	21m 38s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 23s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 42s	the patch passed
+1 💚	compile	9m 55s	the patch passed
+1 💚	cc	9m 55s	the patch passed
+1 💚	javac	9m 55s	the patch passed
+1 💚	checkstyle	2m 25s	root: The patch generated 0 new + 800 unchanged - 33 fixed = 800 total (was 833)
+1 💚	rubocop	0m 6s	The patch generated 0 new + 3 unchanged - 8 fixed = 3 total (was 11)
+1 💚	shellcheck	0m 3s	There were no new shellcheck issues.
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	xml	0m 21s	The patch has no ill-formed XML file.
+0 🆗	refguide	3m 44s	patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
+1 💚	hadoopcheck	12m 24s	Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚	hbaseprotoc	13m 39s	the patch passed
+1 💚	spotbugs	22m 32s	the patch passed
		_ Other Tests _
+1 💚	asflicense	3m 17s	The patch does not generate ASF License warnings.
		135m 3s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	dupname asflicense shellcheck shelldocs javac hadoopcheck xml compile spotbugs hbaseanti checkstyle cc hbaseprotoc prototool rubocop refguide
uname	Linux f947ce28dd12 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `8be2ad2`
Default Java	AdoptOpenJDK-1.8.0_282-b08
refguide	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/artifact/yetus-general-check/output/branch-site/book.html
refguide	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/artifact/yetus-general-check/output/patch-site/book.html
Max. process+thread count	141 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/console
versions	git=2.17.1 maven=3.6.3 shellcheck=0.4.6 spotbugs=4.2.2 rubocop=0.80.0
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-31T20:23:23Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	3m 51s	Docker mode activated.
-0 ⚠️	yetus	0m 7s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 26s	Maven dependency ordering for branch
+1 💚	mvninstall	4m 24s	branch-2 passed
+1 💚	compile	3m 0s	branch-2 passed
+1 💚	shadedjars	6m 59s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 55s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 25s	Maven dependency ordering for patch
+1 💚	mvninstall	4m 17s	the patch passed
+1 💚	compile	3m 3s	the patch passed
+1 💚	javac	3m 3s	the patch passed
+1 💚	shadedjars	7m 16s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	7m 1s	the patch passed
		_ Other Tests _
+1 💚	unit	171m 56s	root in the patch passed.
		224m 40s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux dc4dc9dd33f4 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `8be2ad2`
Default Java	AdoptOpenJDK-11.0.10+9
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/testReport/
Max. process+thread count	6370 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/console
versions	git=2.17.1 maven=3.6.3
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2021-08-31T23:03:52Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	6m 52s	Docker mode activated.
-0 ⚠️	yetus	0m 7s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 32s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 19s	branch-2 passed
+1 💚	compile	2m 12s	branch-2 passed
+1 💚	shadedjars	6m 5s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 5s	branch-2 passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 20s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 21s	the patch passed
+1 💚	compile	2m 15s	the patch passed
+1 💚	javac	2m 15s	the patch passed
+1 💚	shadedjars	6m 29s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	6m 2s	the patch passed
		_ Other Tests _
-1 ❌	unit	336m 41s	root in the patch failed.
		385m 8s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR	#3637
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux 075fcbaa2a6f 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `8be2ad2`
Default Java	AdoptOpenJDK-1.8.0_282-b08
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/testReport/
Max. process+thread count	4341 (vs. ulimit of 12500)
modules	C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-asyncfs hbase-server hbase-mapreduce hbase-shell hbase-it hbase-shaded hbase-shaded/hbase-shaded-client hbase-external-blockcache hbase-assembly hbase-shaded/hbase-shaded-testing-util . U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3637/5/console
versions	git=2.17.1 maven=3.6.3
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

taklwu · 2021-09-01T22:28:59Z

with 3 +1s (@Apache9 @petersomogyi @apurtell ) and 1 +0 (@ndimiduk) from committers, rebase and merge into branch-2

taklwu added 18 commits August 27, 2021 09:41

HBASE-26124BackportHBASE-25373"Remove HTrace completely in code ba…
fc1aa5c
…se and try to make use of OpenTelemetry" to branch-2 (#3529) 1/17 commits of HBASE-22120 Signed-off-by: Peter Somogyi <psomogyi@apache.org>

HBASE-26125BackportHBASE-25401"Add trace support for async call in…
1beffa7
… rpc client" to branch-2 (#3543) 2/17 commits of HBASE-22120 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

HBASE-26127BackportHBASE-23898"Add trace support for simple apis i… (
c96d8fd
#3556) 4/17 commits of HBASE-22120, original commit 805b2ae Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

HBASE-26129BackportHBASE-25481"Add host and port attribute when tr…
e19784c
…acing rpc call at client side" to branch-2 (#3589) 6/17 commits of HBASE-22120, original commit ae2c62f Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

HBASE-26130BackportHBASE-25455"Add trace support for HRegion read/… (
811f089
#3594) 7/17 commits of HBASE-22120, original commit 03e12bf Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

HBASE-26132BackportHBASE-25535"Set span kind to CLIENT in Abstract…
2e2cd4c
…RpcClient" to branch-2 (#3607) 9/17 commits of HBASE-22120, original commit bb8c496 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

HBASE-26134BackportHBASE-25617"Revisit the span names" to branch-2 (…
554285e
…#3618) 11/17 commits of HBASE-22120, original commit 8d68f8c Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

HBASE-26136BackportHBASE-25723"Temporarily remove the trace suppor…
d14ab3e
…t for RegionScanner.next" to branch-2 (#3623) 13/17 commits of HBASE-22120, original commit 7f90c22 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

HBASE-26138BackportHBASE-25733"Upgrade opentelemetry to 1.0.1" to …
f925d32
…branch-2 (#3628) 15/17 commits of HBASE-22120, original commit b714889 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

HBASE-26139BackportHBASE-23762"Add documentation on how to enable …
2c0664e
…and view tracing with OpenTelemetry" to branch-2 (#3629) 16/17 commits of HBASE-22120, original commit be4503d Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org>

HBASE-26140BackportHBASE-25778"The tracinig implementation for Asy…
275de8d
…ncConnectionImpl.getHbck is incorrect" to branch-2 (#3631) 17/17 commits of HBASE-22120, original commit f36e153 Co-authored-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Peter Somogyi <psomogyi@apache.org>

taklwu requested review from Apache9, apurtell, petersomogyi and saintstack August 27, 2021 16:46

taklwu merged commit 625093f into branch-2Sep 1, 2021

taklwu deleted the HBASE-25853 branch September 1, 2021 22:30

Uh oh!

Conversation

taklwu commented Aug 27, 2021• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Apache-HBase commented Aug 27, 2021

Uh oh!

Apache-HBase commented Aug 27, 2021

Uh oh!

Apache-HBase commented Aug 27, 2021

Uh oh!

taklwu commented Aug 28, 2021• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Apache-HBase commented Aug 28, 2021

Uh oh!

Apache-HBase commented Aug 28, 2021

Uh oh!

Apache-HBase commented Aug 29, 2021

Uh oh!

Apache9 commented Aug 29, 2021

Uh oh!

taklwu commented Aug 30, 2021

Uh oh!

Apache9 commented Aug 30, 2021• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

taklwu commented Aug 30, 2021

Uh oh!

taklwu commented Aug 30, 2021

Uh oh!

Apache-HBase commented Aug 30, 2021

Uh oh!

Apache-HBase commented Aug 30, 2021

Uh oh!

Apache-HBase commented Aug 30, 2021

Uh oh!

taklwu commented Aug 30, 2021

Uh oh!

Apache-HBase commented Aug 31, 2021

Uh oh!

Apache-HBase commented Aug 31, 2021

Uh oh!

Apache-HBase commented Aug 31, 2021

Uh oh!

Apache-HBase commented Aug 31, 2021

Uh oh!

Apache-HBase commented Aug 31, 2021

Uh oh!

Apache-HBase commented Aug 31, 2021

Uh oh!

taklwu commented Sep 1, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

taklwu commented Aug 27, 2021•
edited
Loading

taklwu commented Aug 28, 2021•
edited
Loading

Apache9 commented Aug 30, 2021•
edited
Loading