Skip to content

HBASE-23246 Fix error prone warning in TestMetricsUserSourceImpl - #789

Merged
apurtell merged 1 commit into
apache:branch-1from
apurtell:HBASE-23246-branch-1
Nov 4, 2019
Merged

HBASE-23246 Fix error prone warning in TestMetricsUserSourceImpl#789
apurtell merged 1 commit into
apache:branch-1from
apurtell:HBASE-23246-branch-1

Conversation

@apurtell

Copy link
Copy Markdown
Contributor

The self comparison check in TestMetricsUserSourceImpl triggers error-prone and is of little value, remove it.

@apurtell
apurtell requested a review from busbeyNovember 2, 2019 18:02
@apurtell

Copy link
Copy Markdown
ContributorAuthor

Thanks for reporting this @busbey

@apurtell

Copy link
Copy Markdown
ContributorAuthor

The error-prone result is:

TestMetricsUserSourceImpl.java:[50,29] [SelfComparison] An object is compared to itself

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
💙reexec0m 32sDocker mode activated.
_ Prechecks _
💚dupname0m 0sNo case conflicting files found.
💚hbaseanti0m 0sPatch does not have any anti-patterns.
💚@author0m 0sThe patch does not contain any @author tags.
💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ branch-1 Compile Tests _
💚mvninstall8m 29sbranch-1 passed
💚compile0m 14sbranch-1 passed with JDK v1.8.0_232
💚compile0m 17sbranch-1 passed with JDK v1.7.0_242
💚checkstyle0m 18sbranch-1 passed
💚shadedjars3m 10sbranch has no errors when building our shaded downstream artifacts.
💚javadoc0m 19sbranch-1 passed with JDK v1.8.0_232
💚javadoc0m 16sbranch-1 passed with JDK v1.7.0_242
💙spotbugs0m 50sUsed deprecated FindBugs config; considering switching to SpotBugs.
💚findbugs0m 46sbranch-1 passed
_ Patch Compile Tests _
💚mvninstall2m 5sthe patch passed
💚compile0m 14sthe patch passed with JDK v1.8.0_232
💚javac0m 14sthe patch passed
💚compile0m 17sthe patch passed with JDK v1.7.0_242
💚javac0m 17sthe patch passed
💚checkstyle0m 14sthe patch passed
💚whitespace0m 0sThe patch has no whitespace issues.
💚shadedjars3m 5spatch has no errors when building our shaded downstream artifacts.
💚hadoopcheck5m 15sPatch does not cause any errors with Hadoop 2.8.5 2.9.2.
💚javadoc0m 13sthe patch passed with JDK v1.8.0_232
💚javadoc0m 17sthe patch passed with JDK v1.7.0_242
💚findbugs0m 50sthe patch passed
_ Other Tests _
💚unit0m 35shbase-hadoop2-compat in the patch passed.
💚asflicense0m 15sThe patch does not generate ASF License warnings.
31m 26s
SubsystemReport/Notes
DockerClient=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-789/1/artifact/out/Dockerfile
GITHUB PR#789
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 7b06f8a753de 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 05:24:09 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-789/out/precommit/personality/provided.sh
git revisionbranch-1 / 2451023
Default Java1.7.0_242
Multi-JDK versions/usr/lib/jvm/zulu-8-amd64:1.8.0_232 /usr/lib/jvm/zulu-7-amd64:1.7.0_242
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-789/1/testReport/
Max. process+thread count77 (vs. ulimit of 10000)
modulesC: hbase-hadoop2-compat U: hbase-hadoop2-compat
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-789/1/console
versionsgit=1.9.1 maven=3.0.5 findbugs=3.0.1
Powered byApache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.

@guangxuChengguangxuCheng left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

@apurtell
apurtell merged commit 3f9ce86 into apache:branch-1Nov 4, 2019
@apurtell
apurtell deleted the HBASE-23246-branch-1 branch November 4, 2019 18:08
@apurtell

Copy link
Copy Markdown
ContributorAuthor

Thanks for the reviews!

Sign up for freeto 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.

5 participants

@apurtell@Apache-HBase@busbey@guangxuCheng@virajjasani