Hello Jean-Daniel Cryans, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3386 to look at the new patch set (#5). Change subject: [java-client] use tablet cache for locateTablet calls ...................................................................... [java-client] use tablet cache for locateTablet calls This simplifies and improves the performance of AsyncKuduClient::locateTable by using the client's existing tablet cache. This change is motivated by an upcoming fix to the locate table logic, which will be easier without the code duplication. The public KuduTable::locateTable methods have been deprecated since they rely on callers having knowledge of the private internals of Kudu's partitioning implementation. Additionally, the methods have been changed to take an exclusive end partition key instead of an inclusive end partition key. Whether the key was inclusive or exclusive was previously undocumented, and the only known caller (the internal ScanToken implementation) was using it as exclusive. Users of these methods are encouraged to use the ScanToken API instead, since it's easier to use and does not leak internal Kudu implementation details. Change-Id: I78b5d400778547a9ee090111663677901dbadd98 --- M docs/release_notes.adoc M java/kudu-client/src/main/java/org/kududb/client/AsyncKuduClient.java M java/kudu-client/src/main/java/org/kududb/client/KuduTable.java M java/kudu-client/src/main/java/org/kududb/client/LocatedTablet.java M java/kudu-client/src/test/java/org/kududb/client/TestKuduTable.java 5 files changed, 134 insertions(+), 93 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/86/3386/5 -- To view, visit http://gerrit.cloudera.org:8080/3386 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I78b5d400778547a9ee090111663677901dbadd98 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins