Skip to content

HBASE-22886 Code Coverage Improvement: Create Unit Tests for class ConnectionId - #541

Merged
HorizonNet merged 3 commits into
apache:masterfrom
rabi-kumar:HBASE-22886
Sep 2, 2019
Merged

HBASE-22886 Code Coverage Improvement: Create Unit Tests for class ConnectionId#541
HorizonNet merged 3 commits into
apache:masterfrom
rabi-kumar:HBASE-22886

Conversation

@rabi-kumar

Copy link
Copy Markdown
Contributor

HBASE-22886 Code Coverage Improvement: Create Unit Tests for class ConnectionId

@HorizonNetHorizonNet changed the title Hbase 22886HBASE-22886 Code Coverage Improvement: Create Unit Tests for class ConnectionIdAug 26, 2019
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec37Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall322master passed
+1compile26master passed
+1checkstyle33master passed
+1shadedjars271branch has no errors when building our shaded downstream artifacts.
+1javadoc24master passed
0spotbugs70Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs67master passed
_ Patch Compile Tests _
+1mvninstall299the patch passed
+1compile26the patch passed
+1javac26the patch passed
+1checkstyle32the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars295patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck924Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc24the patch passed
+1findbugs82the patch passed
_ Other Tests _
+1unit116hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
2990
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/1/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 72f80246dcb0 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / 1ea5d8b
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/1/testReport/
Max. process+thread count293 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/1/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec37Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall322master passed
+1compile25master passed
+1checkstyle33master passed
+1shadedjars270branch has no errors when building our shaded downstream artifacts.
+1javadoc24master passed
0spotbugs71Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs68master passed
_ Patch Compile Tests _
+1mvninstall293the patch passed
+1compile26the patch passed
+1javac26the patch passed
+1checkstyle32the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars302patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck965Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc22the patch passed
+1findbugs71the patch passed
_ Other Tests _
+1unit109hbase-client in the patch passed.
+1asflicense14The patch does not generate ASF License warnings.
2980
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/2/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux d63915a20824 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / 0438fdc
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/2/testReport/
Max. process+thread count291 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/2/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec54Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall353master passed
+1compile28master passed
+1checkstyle34master passed
+1shadedjars279branch has no errors when building our shaded downstream artifacts.
+1javadoc24master passed
0spotbugs74Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs73master passed
_ Patch Compile Tests _
+1mvninstall302the patch passed
+1compile25the patch passed
+1javac25the patch passed
+1checkstyle31the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars272patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck994Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc24the patch passed
+1findbugs84the patch passed
_ Other Tests _
+1unit118hbase-client in the patch passed.
+1asflicense12The patch does not generate ASF License warnings.
3087
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/3/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux f0afccca517c 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / ec68bf3
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/3/testReport/
Max. process+thread count291 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/3/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec36Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall325master passed
+1compile26master passed
+1checkstyle31master passed
+1shadedjars274branch has no errors when building our shaded downstream artifacts.
+1javadoc26master passed
0spotbugs71Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs70master passed
_ Patch Compile Tests _
+1mvninstall295the patch passed
+1compile25the patch passed
+1javac25the patch passed
+1checkstyle32the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars270patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck918Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc23the patch passed
+1findbugs70the patch passed
_ Other Tests _
+1unit109hbase-client in the patch passed.
+1asflicense14The patch does not generate ASF License warnings.
2919
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/4/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 663d2516a891 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / 5106f28
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/4/testReport/
Max. process+thread count295 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/4/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec50Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti1Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall420master passed
+1compile29master passed
+1checkstyle38master passed
+1shadedjars351branch has no errors when building our shaded downstream artifacts.
+1javadoc27master passed
0spotbugs86Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs84master passed
_ Patch Compile Tests _
+1mvninstall380the patch passed
+1compile28the patch passed
+1javac28the patch passed
+1checkstyle35the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars351patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1185Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc24the patch passed
+1findbugs90the patch passed
_ Other Tests _
+1unit122hbase-client in the patch passed.
+1asflicense15The patch does not generate ASF License warnings.
3707
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/5/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 798262e8c916 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / 56980fb
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/5/testReport/
Max. process+thread count289 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/5/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec35Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall343master passed
+1compile26master passed
+1checkstyle33master passed
+1shadedjars288branch has no errors when building our shaded downstream artifacts.
+1javadoc25master passed
0spotbugs71Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs69master passed
_ Patch Compile Tests _
+1mvninstall298the patch passed
+1compile27the patch passed
+1javac27the patch passed
+1checkstyle30the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars273patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck986Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc24the patch passed
+1findbugs76the patch passed
_ Other Tests _
+1unit116hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3033
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/6/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux ba45ba6e9136 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / 56980fb
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/6/testReport/
Max. process+thread count291 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/6/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec78Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall373master passed
+1compile25master passed
+1checkstyle34master passed
+1shadedjars300branch has no errors when building our shaded downstream artifacts.
+1javadoc24master passed
0spotbugs78Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs76master passed
_ Patch Compile Tests _
+1mvninstall341the patch passed
+1compile26the patch passed
+1javac26the patch passed
+1checkstyle33the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars304patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1038Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc22the patch passed
+1findbugs83the patch passed
_ Other Tests _
+1unit104hbase-client in the patch passed.
+1asflicense12The patch does not generate ASF License warnings.
3296
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/7/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 8c762a06e6e2 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / 83e7794
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/7/testReport/
Max. process+thread count262 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/7/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec32Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall357master passed
+1compile25master passed
+1checkstyle33master passed
+1shadedjars305branch has no errors when building our shaded downstream artifacts.
+1javadoc23master passed
0spotbugs76Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs74master passed
_ Patch Compile Tests _
+1mvninstall337the patch passed
+1compile28the patch passed
+1javac28the patch passed
+1checkstyle34the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars303patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1249Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc21the patch passed
+1findbugs78the patch passed
_ Other Tests _
+1unit112hbase-client in the patch passed.
+1asflicense12The patch does not generate ASF License warnings.
3421
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/8/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux e30f2ddeb9f8 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / f6a4c66
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/8/testReport/
Max. process+thread count263 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/8/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec41Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall339master passed
+1compile27master passed
+1checkstyle32master passed
+1shadedjars281branch has no errors when building our shaded downstream artifacts.
+1javadoc25master passed
0spotbugs71Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs68master passed
_ Patch Compile Tests _
+1mvninstall301the patch passed
+1compile27the patch passed
+1javac27the patch passed
+1checkstyle31the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars289patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck941Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc23the patch passed
+1findbugs74the patch passed
_ Other Tests _
+1unit108hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
2992
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/9/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 6357471d1a2f 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / 090c55f
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/9/testReport/
Max. process+thread count295 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/9/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec53Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall427master passed
+1compile29master passed
+1checkstyle39master passed
+1shadedjars342branch has no errors when building our shaded downstream artifacts.
+1javadoc27master passed
0spotbugs85Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs83master passed
_ Patch Compile Tests _
+1mvninstall329the patch passed
+1compile25the patch passed
+1javac25the patch passed
+1checkstyle30the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars282patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck967Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc22the patch passed
+1findbugs75the patch passed
_ Other Tests _
+1unit111hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3252
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/10/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux be9af7fd2ba4 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / ee5cf0d
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/10/testReport/
Max. process+thread count289 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/10/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec38Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall416master passed
+1compile29master passed
+1checkstyle42master passed
+1shadedjars350branch has no errors when building our shaded downstream artifacts.
+1javadoc31master passed
0spotbugs96Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs93master passed
_ Patch Compile Tests _
+1mvninstall387the patch passed
+1compile30the patch passed
+1javac30the patch passed
+1checkstyle40the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars365patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1238Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc28the patch passed
+1findbugs98the patch passed
_ Other Tests _
+1unit128hbase-client in the patch passed.
+1asflicense14The patch does not generate ASF License warnings.
3787
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/11/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 6ad5c16089f5 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / b642ee0
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/11/testReport/
Max. process+thread count263 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/11/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec76Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall364master passed
+1compile25master passed
+1checkstyle36master passed
+1shadedjars313branch has no errors when building our shaded downstream artifacts.
+1javadoc29master passed
0spotbugs90Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs87master passed
_ Patch Compile Tests _
+1mvninstall344the patch passed
+1compile29the patch passed
+1javac29the patch passed
+1checkstyle36the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars306patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1067Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc21the patch passed
+1findbugs83the patch passed
_ Other Tests _
+1unit117hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3355
SubsystemReport/Notes
DockerClient=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/12/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux f1e94474c174 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / 97fbaa6
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/12/testReport/
Max. process+thread count257 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/12/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec34Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall370master passed
+1compile29master passed
+1checkstyle39master passed
+1shadedjars308branch has no errors when building our shaded downstream artifacts.
+1javadoc25master passed
0spotbugs77Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs74master passed
_ Patch Compile Tests _
+1mvninstall334the patch passed
+1compile25the patch passed
+1javac25the patch passed
+1checkstyle33the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars304patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1048Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc28the patch passed
+1findbugs97the patch passed
_ Other Tests _
+1unit125hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3306
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/13/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 3677b57afea5 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / 345c21d
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/13/testReport/
Max. process+thread count262 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/13/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec38Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall405master passed
+1compile30master passed
+1checkstyle43master passed
+1shadedjars365branch has no errors when building our shaded downstream artifacts.
+1javadoc29master passed
0spotbugs98Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs95master passed
_ Patch Compile Tests _
+1mvninstall402the patch passed
+1compile38the patch passed
+1javac38the patch passed
+1checkstyle47the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars311patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1053Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc21the patch passed
+1findbugs80the patch passed
_ Other Tests _
+1unit107hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3502
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/14/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 909b404469ce 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / a5ef6b2
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/14/testReport/
Max. process+thread count263 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/14/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec43Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall411master passed
+1compile28master passed
+1checkstyle39master passed
+1shadedjars336branch has no errors when building our shaded downstream artifacts.
+1javadoc26master passed
0spotbugs85Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs82master passed
_ Patch Compile Tests _
+1mvninstall415the patch passed
+1compile28the patch passed
+1javac28the patch passed
+1checkstyle37the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars347patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1210Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc25the patch passed
+1findbugs86the patch passed
_ Other Tests _
+1unit128hbase-client in the patch passed.
+1asflicense15The patch does not generate ASF License warnings.
3728
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/15/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 551a230f13cb 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / d32bf8d
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/15/testReport/
Max. process+thread count292 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/15/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@HorizonNetHorizonNet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall. Left a few minor points. In general, for me it wasn't clear when you're using xxxSame or when you're using xxxEquals.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec59Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1mvninstall413master passed
+1compile29master passed
+1checkstyle38master passed
+1shadedjars349branch has no errors when building our shaded downstream artifacts.
+1javadoc27master passed
0spotbugs89Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs86master passed
_ Patch Compile Tests _
+1mvninstall385the patch passed
+1compile28the patch passed
+1javac28the patch passed
+1checkstyle37the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars334patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1154Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc24the patch passed
+1findbugs75the patch passed
_ Other Tests _
+1unit111hbase-client in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3544
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/16/artifact/out/Dockerfile
GITHUB PR#541
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 3b50c8ee4ad5 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-541/out/precommit/personality/provided.sh
git revisionmaster / 8e8bd8b
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/16/testReport/
Max. process+thread count286 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-541/16/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@rabi-kumar

Copy link
Copy Markdown
ContributorAuthor

@HorizonNet Thankyou for the suggestions. I have made the necessary changes.

@HorizonNetHorizonNet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, will commit it soon.

@HorizonNet
HorizonNet merged commit 43e732c into apache:masterSep 2, 2019
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

@rabi-kumar@Apache-HBase@HorizonNet