HBASE-21714 Deprecated isTableAvailableWithSplit method in thrift module

[nyl3532016]

No description provided.

[Apache-HBase]

💔 -1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	300	Docker mode activated.
		_ Prechecks _
+1	hbaseanti	0	Patch does not have any anti-patterns.
+1	@author	0	The patch does not contain any @author tags.
-0	test4tests	0	The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
		_ master Compile Tests _
+1	mvninstall	324	master passed
+1	compile	41	master passed
+1	checkstyle	39	master passed
+1	shadedjars	337	branch has no errors when building our shaded downstream artifacts.
+1	findbugs	98	master passed
+1	javadoc	36	master passed
		_ Patch Compile Tests _
+1	mvninstall	310	the patch passed
+1	compile	39	the patch passed
+1	javac	39	the patch passed
-1	checkstyle	37	hbase-thrift: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1	whitespace	0	The patch has no whitespace issues.
+1	shadedjars	344	patch has no errors when building our shaded downstream artifacts.
+1	hadoopcheck	656	Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1	findbugs	118	the patch passed
+1	javadoc	37	the patch passed
		_ Other Tests _
+1	unit	291	hbase-thrift in the patch passed.
+1	asflicense	13	The patch does not generate ASF License warnings.
		3101

Subsystem	Report/Notes
Docker	Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/1/artifact/out/Dockerfile
GITHUB PR	#217
Optional Tests	dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname	Linux 75de4cef6954 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/patchprocess/precommit/personality/provided.sh
git revision	master / e884a25
maven	version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java	1.8.0_181
findbugs	v3.1.11
checkstyle	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/1/artifact/out/diff-checkstyle-hbase-thrift.txt
Test Results	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/1/testReport/
Max. process+thread count	1775 (vs. ulimit of 10000)
modules	C: hbase-thrift U: hbase-thrift
Console output	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/1/console
Powered by	Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

[Apache9]

Should be 2.3.0 or 2.2.0? And please add comment to say when will we remove it, I would like to remove it in 3.0.0, to align with the Table interface. So here we need to tell user that we will remove it in 3.0.0 for aligning with the Table interface, as in general, it should be removed in 4.0.0.

[Apache9]

@nyl3532016 You can reuse the old PR actually. not a big problem anyway.

[Apache-HBase]

💔 -1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	249	Docker mode activated.
		_ Prechecks _
+1	hbaseanti	0	Patch does not have any anti-patterns.
+1	@author	0	The patch does not contain any @author tags.
-0	test4tests	0	The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
		_ master Compile Tests _
+1	mvninstall	267	master passed
+1	compile	34	master passed
+1	checkstyle	30	master passed
+1	shadedjars	249	branch has no errors when building our shaded downstream artifacts.
+1	findbugs	78	master passed
+1	javadoc	28	master passed
		_ Patch Compile Tests _
+1	mvninstall	244	the patch passed
+1	compile	33	the patch passed
+1	javac	33	the patch passed
-1	checkstyle	29	hbase-thrift: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1	whitespace	0	The patch has no whitespace issues.
+1	shadedjars	256	patch has no errors when building our shaded downstream artifacts.
+1	hadoopcheck	463	Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1	findbugs	89	the patch passed
+1	javadoc	28	the patch passed
		_ Other Tests _
+1	unit	278	hbase-thrift in the patch passed.
+1	asflicense	9	The patch does not generate ASF License warnings.
		2438

Subsystem	Report/Notes
Docker	Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/2/artifact/out/Dockerfile
GITHUB PR	#217
Optional Tests	dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname	Linux 5973013fe39c 4.4.0-137-generic #163-Ubuntu SMP Mon Sep 24 13:14:43 UTC 2018 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/patchprocess/precommit/personality/provided.sh
git revision	master / e884a25
maven	version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java	1.8.0_181
findbugs	v3.1.11
checkstyle	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/2/artifact/out/diff-checkstyle-hbase-thrift.txt
Test Results	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/2/testReport/
Max. process+thread count	1783 (vs. ulimit of 10000)
modules	C: hbase-thrift U: hbase-thrift
Console output	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/2/console
Powered by	Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

[Apache-HBase]

💔 -1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	47	Docker mode activated.
		_ Prechecks _
+1	hbaseanti	0	Patch does not have any anti-patterns.
+1	@author	0	The patch does not contain any @author tags.
-0	test4tests	0	The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
		_ master Compile Tests _
+1	mvninstall	237	master passed
+1	compile	29	master passed
+1	checkstyle	28	master passed
+1	shadedjars	249	branch has no errors when building our shaded downstream artifacts.
+1	findbugs	77	master passed
+1	javadoc	28	master passed
		_ Patch Compile Tests _
+1	mvninstall	245	the patch passed
+1	compile	30	the patch passed
+1	javac	30	the patch passed
-1	checkstyle	29	hbase-thrift: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1	whitespace	0	The patch has no whitespace issues.
+1	shadedjars	245	patch has no errors when building our shaded downstream artifacts.
+1	hadoopcheck	495	Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1	findbugs	87	the patch passed
+1	javadoc	31	the patch passed
		_ Other Tests _
+1	unit	283	hbase-thrift in the patch passed.
+1	asflicense	11	The patch does not generate ASF License warnings.
		2379

Subsystem	Report/Notes
Docker	Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/3/artifact/out/Dockerfile
GITHUB PR	#217
Optional Tests	dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname	Linux 4d3da1020c7b 4.4.0-137-generic #163-Ubuntu SMP Mon Sep 24 13:14:43 UTC 2018 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/patchprocess/precommit/personality/provided.sh
git revision	master / e884a25
maven	version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java	1.8.0_181
findbugs	v3.1.11
checkstyle	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/3/artifact/out/diff-checkstyle-hbase-thrift.txt
Test Results	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/3/testReport/
Max. process+thread count	1778 (vs. ulimit of 10000)
modules	C: hbase-thrift U: hbase-thrift
Console output	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/3/console
Powered by	Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

[Apache-HBase]

💔 -1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	39	Docker mode activated.
		_ Prechecks _
+1	hbaseanti	0	Patch does not have any anti-patterns.
+1	@author	0	The patch does not contain any @author tags.
-0	test4tests	0	The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
		_ master Compile Tests _
+1	mvninstall	242	master passed
+1	compile	33	master passed
+1	checkstyle	33	master passed
+1	shadedjars	273	branch has no errors when building our shaded downstream artifacts.
+1	findbugs	76	master passed
+1	javadoc	29	master passed
		_ Patch Compile Tests _
+1	mvninstall	234	the patch passed
+1	compile	31	the patch passed
+1	javac	31	the patch passed
-1	checkstyle	29	hbase-thrift: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1	whitespace	0	The patch has no whitespace issues.
+1	shadedjars	248	patch has no errors when building our shaded downstream artifacts.
+1	hadoopcheck	520	Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1	findbugs	87	the patch passed
+1	javadoc	29	the patch passed
		_ Other Tests _
+1	unit	265	hbase-thrift in the patch passed.
+1	asflicense	11	The patch does not generate ASF License warnings.
		2253

Subsystem	Report/Notes
Docker	Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/4/artifact/out/Dockerfile
GITHUB PR	#217
Optional Tests	dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname	Linux 8739f86a9063 4.4.0-137-generic #163-Ubuntu SMP Mon Sep 24 13:14:43 UTC 2018 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/patchprocess/precommit/personality/provided.sh
git revision	master / e884a25
maven	version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java	1.8.0_181
findbugs	v3.1.11
checkstyle	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/4/artifact/out/diff-checkstyle-hbase-thrift.txt
Test Results	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/4/testReport/
Max. process+thread count	1809 (vs. ulimit of 10000)
modules	C: hbase-thrift U: hbase-thrift
Console output	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/4/console
Powered by	Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

[Apache-HBase]

💔 -1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	39	Docker mode activated.
		_ Prechecks _
+1	hbaseanti	0	Patch does not have any anti-patterns.
+1	@author	0	The patch does not contain any @author tags.
-0	test4tests	0	The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
		_ master Compile Tests _
+1	mvninstall	260	master passed
+1	compile	31	master passed
+1	checkstyle	34	master passed
+1	shadedjars	272	branch has no errors when building our shaded downstream artifacts.
+1	findbugs	80	master passed
+1	javadoc	30	master passed
		_ Patch Compile Tests _
+1	mvninstall	252	the patch passed
+1	compile	31	the patch passed
+1	javac	31	the patch passed
-1	checkstyle	32	hbase-thrift: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1	whitespace	0	The patch has no whitespace issues.
+1	shadedjars	272	patch has no errors when building our shaded downstream artifacts.
+1	hadoopcheck	523	Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1	findbugs	88	the patch passed
+1	javadoc	30	the patch passed
		_ Other Tests _
+1	unit	260	hbase-thrift in the patch passed.
+1	asflicense	10	The patch does not generate ASF License warnings.
		2324

Subsystem	Report/Notes
Docker	Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/5/artifact/out/Dockerfile
GITHUB PR	#217
Optional Tests	dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname	Linux bb0bd2583c78 4.4.0-141-generic #167~14.04.1-Ubuntu SMP Mon Dec 10 13:20:24 UTC 2018 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/patchprocess/precommit/personality/provided.sh
git revision	master / e884a25
maven	version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java	1.8.0_181
findbugs	v3.1.11
checkstyle	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/5/artifact/out/diff-checkstyle-hbase-thrift.txt
Test Results	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/5/testReport/
Max. process+thread count	1925 (vs. ulimit of 10000)
modules	C: hbase-thrift U: hbase-thrift
Console output	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/5/console
Powered by	Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

[nyl3532016]

this checkstyle -1 is a problem?

[HorizonNet]

@nyl3532016 Yes, would be good to get this fixed before committing it. It should be an easy one.

[Apache-HBase]

💔 -1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	39	Docker mode activated.
		_ Prechecks _
+1	hbaseanti	0	Patch does not have any anti-patterns.
+1	@author	0	The patch does not contain any @author tags.
-0	test4tests	0	The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
		_ master Compile Tests _
+1	mvninstall	247	master passed
+1	compile	32	master passed
+1	checkstyle	31	master passed
+1	shadedjars	255	branch has no errors when building our shaded downstream artifacts.
+1	findbugs	78	master passed
+1	javadoc	29	master passed
		_ Patch Compile Tests _
+1	mvninstall	237	the patch passed
+1	compile	32	the patch passed
+1	javac	32	the patch passed
-1	checkstyle	32	hbase-thrift: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1	whitespace	0	The patch has no whitespace issues.
+1	shadedjars	262	patch has no errors when building our shaded downstream artifacts.
+1	hadoopcheck	516	Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1	findbugs	86	the patch passed
+1	javadoc	28	the patch passed
		_ Other Tests _
+1	unit	268	hbase-thrift in the patch passed.
+1	asflicense	10	The patch does not generate ASF License warnings.
		2256

Subsystem	Report/Notes
Docker	Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/6/artifact/out/Dockerfile
GITHUB PR	#217
Optional Tests	dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname	Linux 3ec125d05510 4.4.0-137-generic #163-Ubuntu SMP Mon Sep 24 13:14:43 UTC 2018 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/patchprocess/precommit/personality/provided.sh
git revision	master / e884a25
maven	version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java	1.8.0_181
findbugs	v3.1.11
checkstyle	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/6/artifact/out/diff-checkstyle-hbase-thrift.txt
Test Results	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/6/testReport/
Max. process+thread count	1791 (vs. ulimit of 10000)
modules	C: hbase-thrift U: hbase-thrift
Console output	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/6/console
Powered by	Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

[Apache-HBase]

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	44	Docker mode activated.
		_ Prechecks _
+1	hbaseanti	0	Patch does not have any anti-patterns.
+1	@author	0	The patch does not contain any @author tags.
-0	test4tests	0	The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
		_ master Compile Tests _
+1	mvninstall	260	master passed
+1	compile	30	master passed
+1	checkstyle	32	master passed
+1	shadedjars	272	branch has no errors when building our shaded downstream artifacts.
+1	findbugs	78	master passed
+1	javadoc	30	master passed
		_ Patch Compile Tests _
+1	mvninstall	234	the patch passed
+1	compile	31	the patch passed
+1	javac	31	the patch passed
+1	checkstyle	30	the patch passed
+1	whitespace	1	The patch has no whitespace issues.
+1	shadedjars	261	patch has no errors when building our shaded downstream artifacts.
+1	hadoopcheck	476	Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1	findbugs	88	the patch passed
+1	javadoc	30	the patch passed
		_ Other Tests _
+1	unit	310	hbase-thrift in the patch passed.
+1	asflicense	9	The patch does not generate ASF License warnings.
		2290

Subsystem	Report/Notes
Docker	Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/7/artifact/out/Dockerfile
GITHUB PR	#217
Optional Tests	dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname	Linux 1d019037a1df 4.4.0-137-generic #163-Ubuntu SMP Mon Sep 24 13:14:43 UTC 2018 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/patchprocess/precommit/personality/provided.sh
git revision	master / e884a25
maven	version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java	1.8.0_181
findbugs	v3.1.11
Test Results	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/7/testReport/
Max. process+thread count	1784 (vs. ulimit of 10000)
modules	C: hbase-thrift U: hbase-thrift
Console output	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/7/console
Powered by	Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

[Apache9]

It should be 2.2.0? I mean you need to say that the same method in Table interface has been deprecated since 2.0.0, so we will remove it in 3.0.0 release.

[nyl3532016]

ok ,I have amended

[Apache-HBase]

🎊 +1 overall

Vote	Subsystem	Runtime	Comment
0	reexec	40	Docker mode activated.
		_ Prechecks _
+1	hbaseanti	0	Patch does not have any anti-patterns.
+1	@author	0	The patch does not contain any @author tags.
-0	test4tests	0	The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
		_ master Compile Tests _
+1	mvninstall	239	master passed
+1	compile	32	master passed
+1	checkstyle	31	master passed
+1	shadedjars	252	branch has no errors when building our shaded downstream artifacts.
+1	findbugs	78	master passed
+1	javadoc	29	master passed
		_ Patch Compile Tests _
+1	mvninstall	239	the patch passed
+1	compile	32	the patch passed
+1	javac	32	the patch passed
+1	checkstyle	30	the patch passed
+1	whitespace	0	The patch has no whitespace issues.
+1	shadedjars	262	patch has no errors when building our shaded downstream artifacts.
+1	hadoopcheck	510	Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1	findbugs	87	the patch passed
+1	javadoc	29	the patch passed
		_ Other Tests _
+1	unit	287	hbase-thrift in the patch passed.
+1	asflicense	11	The patch does not generate ASF License warnings.
		2261

Subsystem	Report/Notes
Docker	Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/8/artifact/out/Dockerfile
GITHUB PR	#217
Optional Tests	dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname	Linux f3901508eb13 4.4.0-137-generic #163-Ubuntu SMP Mon Sep 24 13:14:43 UTC 2018 x86_64 GNU/Linux
Build tool	maven
Personality	/testptch/patchprocess/precommit/personality/provided.sh
git revision	master / e884a25
maven	version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java	1.8.0_181
findbugs	v3.1.11
Test Results	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/8/testReport/
Max. process+thread count	1795 (vs. ulimit of 10000)
modules	C: hbase-thrift U: hbase-thrift
Console output	https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-217/8/console
Powered by	Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.