Adar Dembo has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/13431 )
Change subject: java: fix errorprone warnings
......................................................................
java: fix errorprone warnings
kudu-client/src/main/java/org/apache/kudu/client/ListTablesResponse.java:85:
warning: [EqualsGetClass] Overriding Object#equals in a non-final class by
using getClass rather than instanceof breaks substitutability of subclasses.
public boolean equals(Object o) {
^
kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java:118:
warning: [UnusedVariable] The field 'PERCENTAGE_RANGE' is never read.
private static final Range<Float> PERCENTAGE_RANGE = Range.closed(0.0f, 1.0f);
^
(see https://errorprone.info/bugpattern/UnusedVariable)
Did you mean to remove this line or 'static { Range.closed(0.0f, 1.0f); }'?
Change-Id: Ifbd36b2bbd7b591b08661838d68584f152b1aaab
Reviewed-on: http://gerrit.cloudera.org:8080/13431
Reviewed-by: Andrew Wong <[email protected]>
Tested-by: Adar Dembo <[email protected]>
---
M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
M java/kudu-client/src/main/java/org/apache/kudu/client/ListTablesResponse.java
2 files changed, 1 insertion(+), 3 deletions(-)
Approvals:
Andrew Wong: Looks good to me, approved
Adar Dembo: Verified
--
To view, visit http://gerrit.cloudera.org:8080/13431
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifbd36b2bbd7b591b08661838d68584f152b1aaab
Gerrit-Change-Number: 13431
Gerrit-PatchSet: 3
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
