Skip to content

HBASE-22232 Removed deprecated methods in CompareFilter - #144

Merged
HorizonNet merged 1 commit into
apache:masterfrom
ultratendency:HBASE-22232
Apr 23, 2019
Merged

HBASE-22232 Removed deprecated methods in CompareFilter#144
HorizonNet merged 1 commit into
apache:masterfrom
ultratendency:HBASE-22232

Conversation

@HorizonNet

Copy link
Copy Markdown
Contributor

No description provided.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec24Docker mode activated.
_ Prechecks _
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 15 new or modified test files.
_ master Compile Tests _
0mvndep27Maven dependency ordering for branch
+1mvninstall259master passed
+1compile172master passed
+1checkstyle172master passed
+1shadedjars270branch has no errors when building our shaded downstream artifacts.
+1findbugs432master passed
+1javadoc117master passed
_ Patch Compile Tests _
0mvndep14Maven dependency ordering for patch
+1mvninstall262the patch passed
+1compile170the patch passed
+1javac170the patch passed
-1checkstyle34hbase-client: The patch generated 2 new + 188 unchanged - 52 fixed = 190 total (was 240)
-1checkstyle68hbase-server: The patch generated 4 new + 113 unchanged - 12 fixed = 117 total (was 125)
+1checkstyle17hbase-mapreduce: The patch generated 0 new + 47 unchanged - 2 fixed = 47 total (was 49)
+1checkstyle31The patch passed checkstyle in hbase-thrift
+1checkstyle18hbase-rest: The patch generated 0 new + 242 unchanged - 3 fixed = 242 total (was 245)
+1whitespace0The patch has no whitespace issues.
+1shadedjars271patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck524Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1findbugs455the patch passed
+1javadoc121the patch passed
_ Other Tests _
+1unit198hbase-client in the patch passed.
+1unit8531hbase-server in the patch passed.
-1unit958hbase-mapreduce in the patch failed.
+1unit212hbase-thrift in the patch passed.
+1unit284hbase-rest in the patch passed.
+1asflicense175The patch does not generate ASF License warnings.
13960
ReasonTests
Failed junit testshadoop.hbase.snapshot.TestExportSnapshotNoCluster
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/1/artifact/out/Dockerfile
GITHUB PR#144
Optional Testsdupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 3a14f3078894 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / 16146a1
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java1.8.0_181
findbugsv3.1.11
checkstylehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/1/artifact/out/diff-checkstyle-hbase-client.txt
checkstylehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/1/artifact/out/diff-checkstyle-hbase-server.txt
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/1/artifact/out/patch-unit-hbase-mapreduce.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/1/testReport/
Max. process+thread count5431 (vs. ulimit of 10000)
modulesC: hbase-client hbase-server hbase-mapreduce hbase-thrift hbase-rest U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/1/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec0Docker mode activated.
-1patch6#144 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.
SubsystemReport/Notes
GITHUB PR#144
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/2/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec26Docker mode activated.
_ Prechecks _
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 10 new or modified test files.
_ master Compile Tests _
0mvndep22Maven dependency ordering for branch
+1mvninstall259master passed
+1compile138master passed
+1checkstyle153master passed
+1shadedjars280branch has no errors when building our shaded downstream artifacts.
+1findbugs422master passed
+1javadoc106master passed
_ Patch Compile Tests _
0mvndep14Maven dependency ordering for patch
+1mvninstall259the patch passed
+1compile134the patch passed
+1javac134the patch passed
+1checkstyle35hbase-client: The patch generated 0 new + 188 unchanged - 52 fixed = 188 total (was 240)
+1checkstyle68hbase-server: The patch generated 0 new + 94 unchanged - 9 fixed = 94 total (was 103)
+1checkstyle32The patch passed checkstyle in hbase-thrift
+1checkstyle20hbase-rest: The patch generated 0 new + 242 unchanged - 3 fixed = 242 total (was 245)
+1whitespace0The patch has no whitespace issues.
+1shadedjars263patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck498Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1findbugs419the patch passed
+1javadoc108the patch passed
_ Other Tests _
+1unit197hbase-client in the patch passed.
-1unit7606hbase-server in the patch failed.
+1unit211hbase-thrift in the patch passed.
+1unit276hbase-rest in the patch passed.
+1asflicense97The patch does not generate ASF License warnings.
11765
ReasonTests
Failed junit testshadoop.hbase.master.replication.TestRegisterPeerWorkerWhenRestarting
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/3/artifact/out/Dockerfile
GITHUB PR#144
Optional Testsdupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux c118eaacd428 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / 6e95977
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java1.8.0_181
findbugsv3.1.11
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/3/artifact/out/patch-unit-hbase-server.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/3/testReport/
Max. process+thread count4788 (vs. ulimit of 10000)
modulesC: hbase-client hbase-server hbase-thrift hbase-rest U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/3/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec24Docker mode activated.
_ Prechecks _
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 10 new or modified test files.
_ master Compile Tests _
0mvndep23Maven dependency ordering for branch
+1mvninstall240master passed
+1compile133master passed
+1checkstyle156master passed
+1shadedjars265branch has no errors when building our shaded downstream artifacts.
+1findbugs389master passed
+1javadoc101master passed
_ Patch Compile Tests _
0mvndep13Maven dependency ordering for patch
+1mvninstall231the patch passed
+1compile134the patch passed
+1javac134the patch passed
+1checkstyle33hbase-client: The patch generated 0 new + 186 unchanged - 52 fixed = 186 total (was 238)
+1checkstyle66hbase-server: The patch generated 0 new + 90 unchanged - 9 fixed = 90 total (was 99)
+1checkstyle28The patch passed checkstyle in hbase-thrift
+1checkstyle16hbase-rest: The patch generated 0 new + 242 unchanged - 3 fixed = 242 total (was 245)
+1whitespace0The patch has no whitespace issues.
+1shadedjars258patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck501Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1findbugs429the patch passed
+1javadoc101the patch passed
_ Other Tests _
+1unit200hbase-client in the patch passed.
+1unit8249hbase-server in the patch passed.
+1unit223hbase-thrift in the patch passed.
+1unit306hbase-rest in the patch passed.
+1asflicense96The patch does not generate ASF License warnings.
12339
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/4/artifact/out/Dockerfile
GITHUB PR#144
Optional Testsdupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 786bab52670f 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / 2067b23
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java1.8.0_181
findbugsv3.1.11
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/4/testReport/
Max. process+thread count5288 (vs. ulimit of 10000)
modulesC: hbase-client hbase-server hbase-thrift hbase-rest U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-144/4/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@HorizonNet
HorizonNet merged commit 1f0e43a into apache:masterApr 23, 2019
@HorizonNet
HorizonNet deleted the HBASE-22232 branch April 23, 2019 11:25
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@HorizonNet@Apache-HBase@saintstack