Dan Burkert has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/3386

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, 109 insertions(+), 86 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/86/3386/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3386
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I78b5d400778547a9ee090111663677901dbadd98
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <d...@cloudera.com>

Reply via email to