Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10547 )

Change subject: [Java] Add private API to get a table range partitions
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/10547/1/java/kudu-client/src/main/java/org/apache/kudu/client/Partition.java
File java/kudu-client/src/main/java/org/apache/kudu/client/Partition.java:

http://gerrit.cloudera.org:8080/#/c/10547/1/java/kudu-client/src/main/java/org/apache/kudu/client/Partition.java@105
PS1, Line 105:   public PartialRow getDecodedRangeKeyStart(KuduTable table) {
> Can these be default visibility?
They can't because I am not calling them from the same package in the backup 
code.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I612b93fbebc6f2f1c449bb3c7f2e561f1485a720
Gerrit-Change-Number: 10547
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Comment-Date: Thu, 31 May 2018 18:26:42 +0000
Gerrit-HasComments: Yes

Reply via email to