Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Add a method to return rows in order ......................................................................
[java client] Add a method to return rows in order This patch adds an unstable package private API on the scanners to request the rows to be returned sorted by primary key. This is meant to be used by more advanced users that understand what they are doing. It will remain package private until we start internally checking that if the table is hash partitioned, all hash columns have equality predicates. The new test changes how ITScannerMultiTablet behaves. It now inserts more rows and does so randomly, into a hash-partitioned table. That, plus a new config for TS to flush after 1MB, makes it possible to test this new method. FWIW the change to 1MB flush applies to all the Java client tests. It could be made configurable but I haven't found a need for it yet. Change-Id: I865e28bddd945111ee159b6f2715a8629b75743b Reviewed-on: http://gerrit.cloudera.org:8080/2951 Reviewed-by: Jean-Daniel Cryans Tested-by: Kudu Jenkins --- M java/kudu-client/src/main/java/org/kududb/client/AbstractKuduScannerBuilder.java M java/kudu-client/src/main/java/org/kududb/client/AsyncKuduScanner.java M java/kudu-client/src/main/java/org/kududb/client/KuduScanner.java M java/kudu-client/src/test/java/org/kududb/client/ITScannerMultiTablet.java M java/kudu-client/src/test/java/org/kududb/client/MiniKuduCluster.java 5 files changed, 83 insertions(+), 14 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2951 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I865e28bddd945111ee159b6f2715a8629b75743b Gerrit-PatchSet: 7 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]>
