HBASE-24940: runCatalogJanitor() API should return -1 to indicate already running status #2331

arshadmohammad · 2020-08-30T15:31:36Z

No description provided.

Apache-HBase · 2020-08-30T15:33:17Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 0s	Docker mode activated.
-1 ❌	patch	0m 3s	#2331 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem	Report/Notes
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/1/console
versions	git=2.17.1
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2020-08-30T15:33:18Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 0s	Docker mode activated.
-1 ❌	patch	0m 2s	#2331 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem	Report/Notes
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/1/console
versions	git=2.17.1
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2020-08-30T15:33:18Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 0s	Docker mode activated.
-1 ❌	patch	0m 3s	#2331 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem	Report/Notes
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/1/console
versions	git=2.17.1
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

virajjasani

As per the current behaviour, runCatalogJanitor returns 0 if previous run is in progress. By making it return -1, do we want client to wait for certain duration before retrying? Is that the purpose behind this change?
Please resolve build issue.
Thanks

…eady running status

arshadmohammad · 2020-08-31T07:32:13Z

When previous scan is progress return value is 0. This 0 indicates that no items are garbage collected. This is wrong because in actual there was not attempt to garbage collect any item. Purpose of this change is to correct this wrong behavior to inform the user properly so they take correct actions.

By making it return -1, do we want client to wait for certain duration before retrying? Is that the purpose behind this change?

It is up to the API user how they want to handle it. yes, running after time could be one of the options

arshadmohammad · 2020-08-31T07:33:29Z

Re based the PR to resolve build issue

Apache-HBase · 2020-08-31T07:41:04Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	1m 36s	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.
		_ master Compile Tests _
+0 🆗	mvndep	0m 22s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 43s	master passed
+1 💚	checkstyle	1m 31s	master passed
+1 💚	spotbugs	3m 6s	master passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 13s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 22s	the patch passed
+1 💚	checkstyle	1m 29s	the patch passed
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	hadoopcheck	11m 11s	Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚	spotbugs	3m 15s	the patch passed
		_ Other Tests _
+1 💚	asflicense	0m 26s	The patch does not generate ASF License warnings.
		37m 45s

Subsystem	Report/Notes
Docker	Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Optional Tests	dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle
uname	Linux b035459ee21c 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	master / `25fcc40`
Max. process+thread count	94 (vs. ulimit of 12500)
modules	C: hbase-client hbase-server U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/console
versions	git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2020-08-31T11:27:26Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	1m 52s	Docker mode activated.
-0 ⚠️	yetus	0m 2s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ master Compile Tests _
+0 🆗	mvndep	0m 24s	Maven dependency ordering for branch
+1 💚	mvninstall	5m 7s	master passed
+1 💚	compile	1m 52s	master passed
+1 💚	shadedjars	7m 37s	branch has no errors when building our shaded downstream artifacts.
-0 ⚠️	javadoc	0m 32s	hbase-client in master failed.
-0 ⚠️	javadoc	0m 45s	hbase-server in master failed.
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 15s	Maven dependency ordering for patch
+1 💚	mvninstall	4m 43s	the patch passed
+1 💚	compile	1m 47s	the patch passed
+1 💚	javac	1m 47s	the patch passed
+1 💚	shadedjars	7m 37s	patch has no errors when building our shaded downstream artifacts.
-0 ⚠️	javadoc	0m 29s	hbase-client in the patch failed.
-0 ⚠️	javadoc	0m 51s	hbase-server in the patch failed.
		_ Other Tests _
+1 💚	unit	1m 42s	hbase-client in the patch passed.
+1 💚	unit	226m 21s	hbase-server in the patch passed.
		264m 1s

Subsystem	Report/Notes
Docker	Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux c16610b79af3 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	master / `25fcc40`
Default Java	2020-01-14
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/testReport/
Max. process+thread count	3406 (vs. ulimit of 12500)
modules	C: hbase-client hbase-server U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/console
versions	git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2020-08-31T11:36:59Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	1m 52s	Docker mode activated.
-0 ⚠️	yetus	0m 2s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ master Compile Tests _
+0 🆗	mvndep	0m 23s	Maven dependency ordering for branch
+1 💚	mvninstall	4m 15s	master passed
+1 💚	compile	1m 30s	master passed
+1 💚	shadedjars	7m 16s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	1m 2s	master passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 14s	Maven dependency ordering for patch
+1 💚	mvninstall	4m 6s	the patch passed
+1 💚	compile	1m 28s	the patch passed
+1 💚	javac	1m 28s	the patch passed
+1 💚	shadedjars	7m 13s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	1m 3s	the patch passed
		_ Other Tests _
+1 💚	unit	1m 16s	hbase-client in the patch passed.
-1 ❌	unit	239m 44s	hbase-server in the patch failed.
		273m 33s

Subsystem	Report/Notes
Docker	Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux e64c1d850c5b 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	master / `25fcc40`
Default Java	1.8.0_232
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/testReport/
Max. process+thread count	3277 (vs. ulimit of 12500)
modules	C: hbase-client hbase-server U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/2/console
versions	git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

arshadmohammad · 2020-08-31T11:55:17Z

-1 ❌	unit	239m 44s	hbase-server in the patch failed.

Failed test cases not related to PR

virajjasani

+1

Apache-HBase · 2020-08-31T13:30:52Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 29s	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.
		_ master Compile Tests _
+0 🆗	mvndep	0m 22s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 39s	master passed
+1 💚	checkstyle	1m 30s	master passed
+1 💚	spotbugs	2m 58s	master passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 14s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 21s	the patch passed
+1 💚	checkstyle	1m 31s	the patch passed
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	hadoopcheck	11m 9s	Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚	spotbugs	3m 16s	the patch passed
		_ Other Tests _
+1 💚	asflicense	0m 26s	The patch does not generate ASF License warnings.
		36m 9s

Subsystem	Report/Notes
Docker	Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Optional Tests	dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle
uname	Linux 9e1a46ac4adb 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	master / `25fcc40`
Max. process+thread count	94 (vs. ulimit of 12500)
modules	C: hbase-client hbase-server U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/console
versions	git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2020-08-31T15:56:41Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 49s	Docker mode activated.
-0 ⚠️	yetus	0m 2s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ master Compile Tests _
+0 🆗	mvndep	0m 24s	Maven dependency ordering for branch
+1 💚	mvninstall	4m 12s	master passed
+1 💚	compile	1m 31s	master passed
+1 💚	shadedjars	7m 57s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	1m 3s	master passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 15s	Maven dependency ordering for patch
+1 💚	mvninstall	4m 38s	the patch passed
+1 💚	compile	1m 27s	the patch passed
+1 💚	javac	1m 27s	the patch passed
+1 💚	shadedjars	7m 55s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	1m 4s	the patch passed
		_ Other Tests _
+1 💚	unit	1m 10s	hbase-client in the patch passed.
+1 💚	unit	147m 32s	hbase-server in the patch passed.
		181m 58s

Subsystem	Report/Notes
Docker	Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux 54032f4e67f8 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	master / `25fcc40`
Default Java	1.8.0_232
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/testReport/
Max. process+thread count	4524 (vs. ulimit of 12500)
modules	C: hbase-client hbase-server U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/console
versions	git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2020-08-31T17:12:00Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	1m 26s	Docker mode activated.
-0 ⚠️	yetus	0m 2s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ master Compile Tests _
+0 🆗	mvndep	0m 26s	Maven dependency ordering for branch
+1 💚	mvninstall	5m 15s	master passed
+1 💚	compile	1m 59s	master passed
+1 💚	shadedjars	8m 18s	branch has no errors when building our shaded downstream artifacts.
-0 ⚠️	javadoc	0m 31s	hbase-client in master failed.
-0 ⚠️	javadoc	0m 46s	hbase-server in master failed.
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 15s	Maven dependency ordering for patch
+1 💚	mvninstall	5m 5s	the patch passed
+1 💚	compile	1m 59s	the patch passed
+1 💚	javac	1m 59s	the patch passed
+1 💚	shadedjars	8m 43s	patch has no errors when building our shaded downstream artifacts.
-0 ⚠️	javadoc	0m 31s	hbase-client in the patch failed.
-0 ⚠️	javadoc	0m 47s	hbase-server in the patch failed.
		_ Other Tests _
+1 💚	unit	1m 39s	hbase-client in the patch passed.
-1 ❌	unit	217m 41s	hbase-server in the patch failed.
		257m 18s

Subsystem	Report/Notes
Docker	Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux bf9052fbf1b8 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	master / `25fcc40`
Default Java	2020-01-14
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/testReport/
Max. process+thread count	3257 (vs. ulimit of 12500)
modules	C: hbase-client hbase-server U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/3/console
versions	git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

virajjasani · 2020-08-31T17:48:48Z

TestRSGroupsFallback is failing consistently, can you check locally? In the meanwhile, let me trigger another build.

arshadmohammad · 2020-08-31T18:26:24Z

I run TestRSGroupsFallback locally three times. Every time it passed.

Apache-HBase · 2020-08-31T18:27:01Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	1m 3s	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.
		_ master Compile Tests _
+0 🆗	mvndep	0m 24s	Maven dependency ordering for branch
+1 💚	mvninstall	3m 49s	master passed
+1 💚	checkstyle	1m 34s	master passed
+1 💚	spotbugs	2m 57s	master passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 14s	Maven dependency ordering for patch
+1 💚	mvninstall	3m 28s	the patch passed
+1 💚	checkstyle	1m 31s	the patch passed
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	hadoopcheck	11m 12s	Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚	spotbugs	3m 12s	the patch passed
		_ Other Tests _
+1 💚	asflicense	0m 27s	The patch does not generate ASF License warnings.
		37m 18s

Subsystem	Report/Notes
Docker	Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Optional Tests	dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle
uname	Linux 2ecbd1166f60 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	master / `979edfe`
Max. process+thread count	94 (vs. ulimit of 12500)
modules	C: hbase-client hbase-server U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/console
versions	git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2020-08-31T22:00:49Z

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 28s	Docker mode activated.
-0 ⚠️	yetus	0m 3s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ master Compile Tests _
+0 🆗	mvndep	0m 24s	Maven dependency ordering for branch
+1 💚	mvninstall	5m 34s	master passed
+1 💚	compile	2m 4s	master passed
+1 💚	shadedjars	8m 52s	branch has no errors when building our shaded downstream artifacts.
-0 ⚠️	javadoc	0m 36s	hbase-client in master failed.
-0 ⚠️	javadoc	0m 57s	hbase-server in master failed.
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 18s	Maven dependency ordering for patch
+1 💚	mvninstall	5m 39s	the patch passed
+1 💚	compile	2m 6s	the patch passed
+1 💚	javac	2m 6s	the patch passed
+1 💚	shadedjars	8m 29s	patch has no errors when building our shaded downstream artifacts.
-0 ⚠️	javadoc	0m 30s	hbase-client in the patch failed.
-0 ⚠️	javadoc	0m 50s	hbase-server in the patch failed.
		_ Other Tests _
+1 💚	unit	1m 36s	hbase-client in the patch passed.
+1 💚	unit	199m 30s	hbase-server in the patch passed.
		239m 56s

Subsystem	Report/Notes
Docker	Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux f7f7b0221fa0 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	master / `979edfe`
Default Java	2020-01-14
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt
javadoc	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/testReport/
Max. process+thread count	3143 (vs. ulimit of 12500)
modules	C: hbase-client hbase-server U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/console
versions	git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2020-08-31T22:27:10Z

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	1m 54s	Docker mode activated.
-0 ⚠️	yetus	0m 2s	Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
		_ Prechecks _
		_ master Compile Tests _
+0 🆗	mvndep	0m 24s	Maven dependency ordering for branch
+1 💚	mvninstall	4m 19s	master passed
+1 💚	compile	1m 47s	master passed
+1 💚	shadedjars	8m 30s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	1m 7s	master passed
		_ Patch Compile Tests _
+0 🆗	mvndep	0m 15s	Maven dependency ordering for patch
+1 💚	mvninstall	4m 37s	the patch passed
+1 💚	compile	1m 46s	the patch passed
+1 💚	javac	1m 46s	the patch passed
+1 💚	shadedjars	8m 56s	patch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	1m 16s	the patch passed
		_ Other Tests _
+1 💚	unit	1m 34s	hbase-client in the patch passed.
-1 ❌	unit	228m 6s	hbase-server in the patch failed.
		266m 23s

Subsystem	Report/Notes
Docker	Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR	#2331
JIRA Issue	HBASE-24940
Optional Tests	javac javadoc unit shadedjars compile
uname	Linux ac678a9f954d 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	master / `979edfe`
Default Java	1.8.0_232
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/testReport/
Max. process+thread count	3279 (vs. ulimit of 12500)
modules	C: hbase-client hbase-server U: .
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2331/4/console
versions	git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

virajjasani · 2020-09-01T07:16:20Z

It's fine, these are flaky tests. If we can get one more approval, it would be really great.

…eady running status Closes #2331 Signed-off-by: Viraj Jasani <vjasani@apache.org>

…eady running status Closes #2331 Co-authored-by: Viraj Jasani <vjasani@apache.org> Signed-off-by: Viraj Jasani <vjasani@apache.org>

…eady running status Closes apache#2331 Signed-off-by: Viraj Jasani <vjasani@apache.org>

virajjasani reviewed Aug 30, 2020

View reviewed changes

HBASE-24940: runCatalogJanitor() API should return -1 to indicate alr…

101f2c1

…eady running status

arshadmohammad force-pushed the HBASE-24940-master branch from 79732e1 to 101f2c1 Compare August 31, 2020 07:02

virajjasani approved these changes Aug 31, 2020

View reviewed changes

virajjasani closed this in a352706 Sep 3, 2020

virajjasani pushed a commit that referenced this pull request Sep 3, 2020

HBASE-24940: runCatalogJanitor() API should return -1 to indicate alr…

17eeaef

…eady running status Closes #2331 Signed-off-by: Viraj Jasani <vjasani@apache.org>

virajjasani pushed a commit that referenced this pull request Sep 3, 2020

HBASE-24940: runCatalogJanitor() API should return -1 to indicate alr…

dc79b26

…eady running status Closes #2331 Signed-off-by: Viraj Jasani <vjasani@apache.org>

virajjasani pushed a commit that referenced this pull request Sep 3, 2020

HBASE-24940: runCatalogJanitor() API should return -1 to indicate alr…

76db1e1

…eady running status Closes #2331 Co-authored-by: Viraj Jasani <vjasani@apache.org> Signed-off-by: Viraj Jasani <vjasani@apache.org>

clarax pushed a commit to clarax/hbase that referenced this pull request Nov 15, 2020

HBASE-24940: runCatalogJanitor() API should return -1 to indicate alr…

74b601c

…eady running status Closes apache#2331 Signed-off-by: Viraj Jasani <vjasani@apache.org>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

HBASE-24940: runCatalogJanitor() API should return -1 to indicate already running status #2331

HBASE-24940: runCatalogJanitor() API should return -1 to indicate already running status #2331

arshadmohammad commented Aug 30, 2020

Apache-HBase commented Aug 30, 2020

Apache-HBase commented Aug 30, 2020

Apache-HBase commented Aug 30, 2020

virajjasani left a comment

arshadmohammad commented Aug 31, 2020 •

edited

Loading

arshadmohammad commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

arshadmohammad commented Aug 31, 2020

virajjasani left a comment

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

virajjasani commented Aug 31, 2020

arshadmohammad commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

virajjasani commented Sep 1, 2020

HBASE-24940: runCatalogJanitor() API should return -1 to indicate already running status #2331

HBASE-24940: runCatalogJanitor() API should return -1 to indicate already running status #2331

Conversation

arshadmohammad commented Aug 30, 2020

Apache-HBase commented Aug 30, 2020

Apache-HBase commented Aug 30, 2020

Apache-HBase commented Aug 30, 2020

virajjasani left a comment

Choose a reason for hiding this comment

arshadmohammad commented Aug 31, 2020 • edited Loading

arshadmohammad commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

arshadmohammad commented Aug 31, 2020

virajjasani left a comment

Choose a reason for hiding this comment

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

virajjasani commented Aug 31, 2020

arshadmohammad commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

Apache-HBase commented Aug 31, 2020

virajjasani commented Sep 1, 2020

arshadmohammad commented Aug 31, 2020 •

edited

Loading