Jean-Daniel Cryans has posted comments on this change.

Change subject: KUDU-1831. Java client does not check if the primary key 
columns are specified first
......................................................................


Patch Set 1: Code-Review-2

I'm pretty sure we don't want this limitation, see 
https://github.com/cloudera/kudu/commit/7d57b8b9204a0b170fc204cb7aaa98ad995cf635#diff-933a1ca025bc6af95c770d34958fe46a

I wonder why the unit tests are still passing though.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ca2a572801c964331ed65c630db28436fcaf86a
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jun He <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: No

Reply via email to