Skip to content

HBASE-29125 Suppress HE_EQUALS_USE_HASHCODE in org.apache.hadoop.hbas… - #6685

Merged
stoty merged 2 commits into
apache:masterfrom
stoty:HBASE-29125
Feb 12, 2025
Merged

HBASE-29125 Suppress HE_EQUALS_USE_HASHCODE in org.apache.hadoop.hbas…#6685
stoty merged 2 commits into
apache:masterfrom
stoty:HBASE-29125

Conversation

@stoty

Copy link
Copy Markdown
Contributor

…e.rest.model.ScannerModel$IncludeStartRowFilter

…e.rest.model.ScannerModel$IncludeStartRowFilter
@stoty
stoty requested a review from Apache9February 12, 2025 10:46
@Apache-HBase

This comment has been minimized.

@Apache9Apache9 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.

+1 for now.

I'm not very familiar with the serialization framework we use here, are there any ways to not use the equals method?

@Apache-HBase

This comment has been minimized.

@stoty

Copy link
Copy Markdown
ContributorAuthor

+1 for now.

I'm not very familiar with the serialization framework we use here, are there any ways to not use the equals method?

No, equals is the whole and only point of this class.
The framework never calls hashCode, we've had a long discussion about this on the original PR, and supression was best solution.

@stoty

Copy link
Copy Markdown
ContributorAuthor

The suppression was added and is discussed in #6499

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 39sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+1 💚mvninstall3m 45smaster passed
+1 💚compile0m 31smaster passed
+1 💚javadoc0m 21smaster passed
+1 💚shadedjars6m 55sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall3m 45sthe patch passed
+1 💚compile0m 27sthe patch passed
+1 💚javac0m 27sthe patch passed
+1 💚javadoc0m 20sthe patch passed
+1 💚shadedjars6m 59spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit4m 21shbase-rest in the patch passed.
29m 8s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6685/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6685
Optional Testsjavac javadoc unit compile shadedjars
unameLinux da1838ae9e44 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 4e83bcf
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6685/2/testReport/
Max. process+thread count1701 (vs. ulimit of 30000)
modulesC: hbase-rest U: hbase-rest
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6685/2/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 29sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
_ master Compile Tests _
+1 💚mvninstall3m 19smaster passed
+1 💚compile0m 33smaster passed
+1 💚checkstyle0m 10smaster passed
-1 ❌spotbugs0m 37s/branch-spotbugs-hbase-rest-warnings.htmlhbase-rest in master has 1 extant spotbugs warnings.
+1 💚spotless0m 45sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚mvninstall3m 5sthe patch passed
+1 💚compile0m 36sthe patch passed
+1 💚javac0m 36sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 10sthe patch passed
+1 💚spotbugs0m 41shbase-rest generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
+1 💚hadoopcheck11m 54sPatch does not cause any errors with Hadoop 3.3.6 3.4.0.
+1 💚spotless0m 44spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 10sThe patch does not generate ASF License warnings.
30m 30s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6685/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6685
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux dbcac46b50c4 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 4e83bcf
Default JavaEclipse Adoptium-17.0.11+9
Max. process+thread count85 (vs. ulimit of 30000)
modulesC: hbase-rest U: hbase-rest
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6685/2/console
versionsgit=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@stoty
stoty merged commit e9869ae into apache:masterFeb 12, 2025
stoty added a commit that referenced this pull request Feb 12, 2025
…e.rest.model.ScannerModel$IncludeStartRowFilter (#6685)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
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.

3 participants

@stoty@Apache-HBase@Apache9