Mahesh Reddy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18589 )

Change subject: [java] KUDU-2671 support adding a range with custom hash schema
......................................................................


Patch Set 9: Code-Review+1

(1 comment)

Besides build failure, lgtm.

http://gerrit.cloudera.org:8080/#/c/18589/9/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java
File java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java:

http://gerrit.cloudera.org:8080/#/c/18589/9/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java@598
PS9, Line 598: assertTrue(String.format("must have bucket %i", i)
There's some build errors relating to the unknown conversion of 'i' here.



--
To view, visit http://gerrit.cloudera.org:8080/18589
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ieaab7a79d4336de7ff6ec84b8c1806407e4fa44e
Gerrit-Change-Number: 18589
Gerrit-PatchSet: 9
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Fri, 01 Jul 2022 18:34:28 +0000
Gerrit-HasComments: Yes

Reply via email to