Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4173 to look at the new patch set (#2). Change subject: [java-client] Add ScanToken.stringifySerializedToken ...................................................................... [java-client] Add ScanToken.stringifySerializedToken stringifySerializedToken takes a serialized scan token, and returns a String suitable for debug printing. The string contains information sufficient to determine which range of tablet(s) the scan token will cover, and which rows within the tablets. Namely, the range partition bounds and primary key bounds. Example output, wrapped for readability (a, b, and c are column names with type STRING): ScanToken{table=org.apache.kudu.client.TestKuduClient-1472595465767, lower-bound-primary-key=(string a=1, string b=3, string c=5), upper-bound-primary-key=(string a=2, string b=4, string c=), hash-partition-buckets: [2], range-partition: [(string a=0, string b=0, string c=0), (string a=3, string b=5, string c=6))} The Java client did not have any method of deserializing encoded primary or partition keys, so most of the work in this commit is introducing utility methods for that purpose. I haven't added tests of the specific format of the strings, but I have added the printing to many of the existing ScanToken tests in order to make sure that the formatting code itself doesn't fail. I've also verified that the output looks good. The format doesn't include information like predicates or consistency, but that could be added in the future if so desired. Change-Id: I42014da270e66c370cc6d6ed68fb38331130ff6d --- M docs/release_notes.adoc M java/kudu-client/src/main/java/org/apache/kudu/client/DeadlineTracker.java M java/kudu-client/src/main/java/org/apache/kudu/client/KeyEncoder.java M java/kudu-client/src/main/java/org/apache/kudu/client/KuduScanToken.java M java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java M java/kudu-client/src/main/java/org/apache/kudu/client/RowResult.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestFlexiblePartitioning.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestKeyEncoding.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestOperation.java M java/kudu-mapreduce/src/main/java/org/apache/kudu/mapreduce/KuduTableInputFormat.java 11 files changed, 457 insertions(+), 81 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/73/4173/2 -- To view, visit http://gerrit.cloudera.org:8080/4173 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I42014da270e66c370cc6d6ed68fb38331130ff6d Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>