amogh-jahagirdar opened a new pull request, #4953:
URL: https://github.com/apache/iceberg/pull/4953
To get rid of some build warnings
`warning: [ObjectsHashCodeUnnecessaryVarargs]
java.util.Objects.hash(non-varargs) should be replaced with
java.util.Objects.hashCode(value) to avoid unnecessary varargs array
allocations.
return Objects.hash(fields);
^
(see
https://github.com/palantir/gradle-baseline#baseline-error-prone-checks)
Did you mean 'return Objects.hashCode(fields)`
In some hashCode implementations we unnecessarily use Objects.hash which
accepts variadic arguments. Under the hood, this will create an array and do
Arrays.hash. However, in these cases since we're passing in a single object
reference, hashCode suffices, and we can avoid ant unnecessary array
allocations,
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
