joonseolee opened a new pull request, #8019:
URL: https://github.com/apache/gravitino/pull/8019
<!--
1. Title: [#<issue>] <type>(<scope>): <subject>
Examples:
- "[#123] feat(operator): support xxx"
- "[#233] fix: check null before access result in xxx"
- "[MINOR] refactor: fix typo in variable name"
- "[MINOR] docs: fix typo in README"
- "[#255] test: fix flaky test NameOfTheTest"
Reference: https://www.conventionalcommits.org/en/v1.0.0/
2. If the PR is unfinished, please mark this PR as draft.
-->
### What changes were proposed in this pull request?
This PR updates the TableEntity.hashCode() implementation to include the
namespace field in the hash calculation.
It also updates related test cases to validate the change and applies
Spotless formatting to comply with the project code style.
### Why are the changes needed?
The current hashCode() implementation does not include the namespace field,
causing objects with different namespaces to have identical hash values.
This could lead to incorrect behavior when TableEntity objects are used in
hash-based collections (e.g., HashSet, HashMap).
By including the namespace, equality and hash consistency are ensured.
Fix: #8012
### Does this PR introduce _any_ user-facing change?
No. This change only affects internal equality/hash logic of TableEntity and
does not modify any user-facing API or property keys.
### How was this patch tested?
Updated existing unit tests in TestEntity.java
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
