Hello 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 (#2).
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. The semantics of
locateTable are changed slightly- the end key is now treated as an
exclusive key instead of inclusive. This is a breaking change for the public
KuduTable::locateTable methods. These methods are a holdover from the pre-scan
token era, and should almost certainly be deprecated. To use correctly they
require great knowledge of Kudu's internal partitioning logic.
I'm not too worried about making this breaking change since the API is pretty
much impossible to use to begin with, and no one is using it (except for the
scan tokens impl, and it was already treating the end key as exclusive).
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, 119 insertions(+), 86 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/86/3386/2
--
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: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
