Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HBASE-22243 Removed deprecated method in Result #152

Merged
merged 1 commit into from
Apr 15, 2019
Merged

HBASE-22243 Removed deprecated method in Result #152

merged 1 commit into from
Apr 15, 2019

Conversation

anis016
Copy link
Contributor

@anis016 anis016 commented Apr 15, 2019

Removed the deprecated method - isPartial() from Result.
Fixes: HBASE-22243

@HorizonNet
Copy link
Contributor

LGTM, pending QA.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 22 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 257 master passed
+1 compile 24 master passed
+1 checkstyle 31 master passed
+1 shadedjars 263 branch has no errors when building our shaded downstream artifacts.
+1 findbugs 63 master passed
+1 javadoc 23 master passed
_ Patch Compile Tests _
+1 mvninstall 241 the patch passed
+1 compile 25 the patch passed
+1 javac 25 the patch passed
+1 checkstyle 30 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 256 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 541 Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1 findbugs 68 the patch passed
+1 javadoc 21 the patch passed
_ Other Tests _
+1 unit 192 hbase-client in the patch passed.
+1 asflicense 10 The patch does not generate ASF License warnings.
2143
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-152/1/artifact/out/Dockerfile
GITHUB PR #152
Optional Tests dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 1040b506616d 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 / 7d1399c
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-152/1/testReport/
Max. process+thread count 291 (vs. ulimit of 10000)
modules C: hbase-client U: hbase-client
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-152/1/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 23 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 25 master passed
+1 checkstyle 28 master passed
+1 shadedjars 260 branch has no errors when building our shaded downstream artifacts.
+1 findbugs 61 master passed
+1 javadoc 23 master passed
_ Patch Compile Tests _
+1 mvninstall 242 the patch passed
+1 compile 26 the patch passed
+1 javac 26 the patch passed
+1 checkstyle 30 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 264 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 502 Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1 findbugs 67 the patch passed
+1 javadoc 21 the patch passed
_ Other Tests _
+1 unit 201 hbase-client in the patch passed.
+1 asflicense 11 The patch does not generate ASF License warnings.
2100
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-152/2/artifact/out/Dockerfile
GITHUB PR #152
Optional Tests dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux d90273c8c702 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 / 7d1399c
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-152/2/testReport/
Max. process+thread count 301 (vs. ulimit of 10000)
modules C: hbase-client U: hbase-client
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-152/2/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@HorizonNet HorizonNet merged commit 88de644 into apache:master Apr 15, 2019
This was referenced Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants