Dan Burkert has posted comments on this change.

Change subject: [java client] Add a method to return rows in order
......................................................................


Patch Set 3: -Code-Review

OK pushed an update that renames the method from unstableSortByPrimaryKey -> 
sortByPrimaryKey, and makes it package private.  It may be made public once we 
want to commit to it, and have internal checks that the scan has equality 
predicates on all hash partitioned columns.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I865e28bddd945111ee159b6f2715a8629b75743b
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to