Hello Jean-Daniel Cryans, 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 (#4).

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, 128 insertions(+), 92 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/86/3386/4
-- 
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: 4
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

Reply via email to