Todd Lipcon has posted comments on this change.

Change subject: [java client] Allow scanners to have projections with 
out-of-order keys
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3015/1/java/kudu-client/src/main/java/org/kududb/Schema.java
File java/kudu-client/src/main/java/org/kududb/Schema.java:

Line 238
> Yeah you're right, although I can't see people using it.
hrm, why are you removing it? it seems useful to me (and in fact more useful 
than getPrimaryKeyColumnCount which will become useless once we support having 
PKs not at the front of the schema)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I550cb255460bb54815d3a30eb973f8c143573e62
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to