HBASE-26775 - add synchronized modifier to the toString() method of ProcedureEvent.java #4681

ryakhovskiy · 2022-08-07T14:22:08Z

No description provided.

…rocedureEvent.java

Apache-HBase · 2022-08-07T14:52:46Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 51s	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 _
+1 💚	mvninstall	2m 51s	branch-2 passed
+1 💚	compile	0m 12s	branch-2 passed
+1 💚	shadedjars	4m 6s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	0m 12s	branch-2 passed
		_ Patch Compile Tests _
+1 💚	mvninstall	2m 29s	the patch passed
+1 💚	compile	0m 12s	the patch passed
+1 💚	javac	0m 12s	the patch passed
+1 💚	shadedjars	4m 0s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	0m 10s	the patch passed
		_ Other Tests _
+1 💚	unit	1m 19s	hbase-procedure in the patch passed.
		17m 30s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#4681
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux 3b23e1f8df40 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `2821cc5`
Default Java	AdoptOpenJDK-11.0.10+9
Test Results	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/1/testReport/
Max. process+thread count	207 (vs. ulimit of 12500)
modules	C: hbase-procedure U: hbase-procedure
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/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 · 2022-08-07T14:54:15Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	1m 14s	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 _
+1 💚	mvninstall	2m 46s	branch-2 passed
+1 💚	compile	0m 15s	branch-2 passed
+1 💚	shadedjars	4m 37s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	0m 14s	branch-2 passed
		_ Patch Compile Tests _
+1 💚	mvninstall	2m 32s	the patch passed
+1 💚	compile	0m 13s	the patch passed
+1 💚	javac	0m 13s	the patch passed
+1 💚	shadedjars	4m 19s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	0m 11s	the patch passed
		_ Other Tests _
+1 💚	unit	1m 28s	hbase-procedure in the patch passed.
		18m 55s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR	#4681
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux b5550764a23d 5.4.0-1071-aws #76~18.04.1-Ubuntu SMP Mon Mar 28 17:49:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `2821cc5`
Default Java	AdoptOpenJDK-1.8.0_282-b08
Test Results	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/1/testReport/
Max. process+thread count	181 (vs. ulimit of 12500)
modules	C: hbase-procedure U: hbase-procedure
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/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 · 2022-08-07T14:55:57Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 23s	Docker mode activated.
		_ Prechecks _
+1 💚	dupname	0m 0s	No case conflicting files found.
+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 _
+1 💚	mvninstall	2m 22s	branch-2 passed
+1 💚	compile	0m 22s	branch-2 passed
+1 💚	checkstyle	0m 11s	branch-2 passed
+1 💚	spotless	0m 45s	branch has no errors when running spotless:check.
+1 💚	spotbugs	0m 23s	branch-2 passed
		_ Patch Compile Tests _
+1 💚	mvninstall	2m 7s	the patch passed
+1 💚	compile	0m 22s	the patch passed
+1 💚	javac	0m 22s	the patch passed
+1 💚	checkstyle	0m 9s	the patch passed
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	hadoopcheck	7m 41s	Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
-1 ❌	spotless	0m 20s	patch has 23 errors when running spotless:check, run spotless:apply to fix.
+1 💚	spotbugs	0m 27s	the patch passed
		_ Other Tests _
+1 💚	asflicense	0m 11s	The patch does not generate ASF License warnings.
		20m 47s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#4681
Optional Tests	dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
uname	Linux 213c8ac04f0f 5.4.0-96-generic #109-Ubuntu SMP Wed Jan 12 16:49:16 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `2821cc5`
Default Java	AdoptOpenJDK-1.8.0_282-b08
spotless	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/1/artifact/yetus-general-check/output/patch-spotless.txt
Max. process+thread count	60 (vs. ulimit of 12500)
modules	C: hbase-procedure U: hbase-procedure
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/1/console
versions	git=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2022-08-07T16:07:16Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 45s	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 _
+1 💚	mvninstall	2m 28s	branch-2 passed
+1 💚	compile	0m 12s	branch-2 passed
+1 💚	shadedjars	4m 3s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	0m 10s	branch-2 passed
		_ Patch Compile Tests _
+1 💚	mvninstall	2m 33s	the patch passed
+1 💚	compile	0m 12s	the patch passed
+1 💚	javac	0m 12s	the patch passed
+1 💚	shadedjars	4m 1s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	0m 10s	the patch passed
		_ Other Tests _
+1 💚	unit	1m 20s	hbase-procedure in the patch passed.
		17m 0s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#4681
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux c05019e1fb7a 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `2821cc5`
Default Java	AdoptOpenJDK-11.0.10+9
Test Results	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/2/testReport/
Max. process+thread count	201 (vs. ulimit of 12500)
modules	C: hbase-procedure U: hbase-procedure
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/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 · 2022-08-07T16:10:52Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	1m 7s	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 _
+1 💚	mvninstall	2m 23s	branch-2 passed
+1 💚	compile	0m 14s	branch-2 passed
+1 💚	shadedjars	4m 58s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	0m 15s	branch-2 passed
		_ Patch Compile Tests _
+1 💚	mvninstall	2m 54s	the patch passed
+1 💚	compile	0m 15s	the patch passed
+1 💚	javac	0m 15s	the patch passed
+1 💚	shadedjars	5m 10s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	0m 14s	the patch passed
		_ Other Tests _
+1 💚	unit	1m 49s	hbase-procedure in the patch passed.
		20m 32s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR	#4681
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux 55283a805318 5.4.0-1071-aws #76~18.04.1-Ubuntu SMP Mon Mar 28 17:49:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `2821cc5`
Default Java	AdoptOpenJDK-1.8.0_282-b08
Test Results	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/2/testReport/
Max. process+thread count	181 (vs. ulimit of 12500)
modules	C: hbase-procedure U: hbase-procedure
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/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 · 2022-08-07T16:10:54Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 24s	Docker mode activated.
		_ Prechecks _
+1 💚	dupname	0m 0s	No case conflicting files found.
+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 _
+1 💚	mvninstall	2m 7s	branch-2 passed
+1 💚	compile	0m 21s	branch-2 passed
+1 💚	checkstyle	0m 10s	branch-2 passed
+1 💚	spotless	0m 40s	branch has no errors when running spotless:check.
+1 💚	spotbugs	0m 20s	branch-2 passed
		_ Patch Compile Tests _
+1 💚	mvninstall	2m 9s	the patch passed
+1 💚	compile	0m 21s	the patch passed
+1 💚	javac	0m 21s	the patch passed
+1 💚	checkstyle	0m 9s	the patch passed
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	hadoopcheck	7m 36s	Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚	spotless	0m 43s	patch has no errors when running spotless:check.
+1 💚	spotbugs	0m 26s	the patch passed
		_ Other Tests _
+1 💚	asflicense	0m 9s	The patch does not generate ASF License warnings.
		20m 40s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#4681
Optional Tests	dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
uname	Linux 4c089a3b1730 5.4.0-96-generic #109-Ubuntu SMP Wed Jan 12 16:49:16 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `2821cc5`
Default Java	AdoptOpenJDK-1.8.0_282-b08
Max. process+thread count	64 (vs. ulimit of 12500)
modules	C: hbase-procedure U: hbase-procedure
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4681/2/console
versions	git=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered by	Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

…rocedureEvent.java (#4681) Signed-off-by: Duo Zhang <[email protected]> (cherry picked from commit 7472661)

…rocedureEvent.java (apache#4681) Signed-off-by: Duo Zhang <[email protected]> (cherry picked from commit 7472661) (cherry picked from commit 0713d03) Change-Id: I833973940c02f0f3cda767c9f4f0231b174c28ed

HBASE-26775 - add synchronized modifier to the toString() method of P…

23242ea

…rocedureEvent.java

HBASE-26775-update formatting

9f68e98

Apache9 approved these changes Aug 7, 2022

View reviewed changes

Apache9 merged commit 7472661 into apache:branch-2 Aug 8, 2022

Apache9 pushed a commit that referenced this pull request Aug 8, 2022

HBASE-26775 - add synchronized modifier to the toString() method of P…

0713d03

…rocedureEvent.java (#4681) Signed-off-by: Duo Zhang <[email protected]> (cherry picked from commit 7472661)

Apache9 pushed a commit that referenced this pull request Aug 8, 2022

HBASE-26775 - add synchronized modifier to the toString() method of P…

948b323

…rocedureEvent.java (#4681) Signed-off-by: Duo Zhang <[email protected]> (cherry picked from commit 7472661)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

HBASE-26775 - add synchronized modifier to the toString() method of ProcedureEvent.java #4681

HBASE-26775 - add synchronized modifier to the toString() method of ProcedureEvent.java #4681

ryakhovskiy commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

HBASE-26775 - add synchronized modifier to the toString() method of ProcedureEvent.java #4681

HBASE-26775 - add synchronized modifier to the toString() method of ProcedureEvent.java #4681

Conversation

ryakhovskiy commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022

Apache-HBase commented Aug 7, 2022