Dan Burkert has posted comments on this change. Change subject: [java-client] implement scan token API ......................................................................
Patch Set 2: (6 comments) http://gerrit.cloudera.org:8080/#/c/2592/2//COMMIT_MSG Commit Message: Line 10: toke > nit Done http://gerrit.cloudera.org:8080/#/c/2592/2/java/kudu-client/src/main/java/org/kududb/client/Bytes.java File java/kudu-client/src/main/java/org/kududb/client/Bytes.java: Line 1105: b.array() > Are you using the array here just to save a copy? A bulk get would do the j Done http://gerrit.cloudera.org:8080/#/c/2592/2/java/kudu-client/src/main/java/org/kududb/client/KuduClient.java File java/kudu-client/src/main/java/org/kududb/client/KuduClient.java: Line 211: * Scan tokens may be used for creating parallel scanners over a Kudu table. > If I was a random user reading this, I'd think it's pretty cool that I can I've changed it to just say it's used for integration with compute frameworks, but I've also added a unit test that does parallel scanning so that we have something to point to when people want to see how it's done. http://gerrit.cloudera.org:8080/#/c/2592/2/java/kudu-client/src/main/java/org/kududb/client/ScanToken.java File java/kudu-client/src/main/java/org/kududb/client/ScanToken.java: Line 36: public class ScanToken implements Comparable<ScanToken> { > Add InterfaceAudience and InterfaceStability, plus javadoc (here and on th Done Line 48: * Returns the tablet which this scan token will scan. > I don't think it's the token that's scanning. Done http://gerrit.cloudera.org:8080/#/c/2592/2/src/kudu/client/CMakeLists.txt File src/kudu/client/CMakeLists.txt: Line 18: PROTOBUF_GENERATE_CPP( > Feels like this file and the next should be in their own commit. Done -- To view, visit http://gerrit.cloudera.org:8080/2592 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I20eff9bf51e893226fc3bc47726565ca62c054e3 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
