Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Extract RemoteTablet from AsyncKuduClient ......................................................................
[java client] Extract RemoteTablet from AsyncKuduClient RemoteTablet is responsible for handling the Java client's view of where replicas are for its tablet, and who the leader is. Extracting this bit of functionality is important if we want to be able to unit test it without bringing up a whole client and possibly a cluster. This patch makes a minor attempt at simplifying the interfaces, with more work to come. Likewise for unit tests. Change-Id: I3d06a573e4307c76a7aebab05cd5238fb0d9a2c6 Reviewed-on: http://gerrit.cloudera.org:8080/4719 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <danburk...@apache.org> --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduScanner.java M java/kudu-client/src/main/java/org/apache/kudu/client/KuduRpc.java M java/kudu-client/src/main/java/org/apache/kudu/client/LocatedTablet.java A java/kudu-client/src/main/java/org/apache/kudu/client/RemoteTablet.java M java/kudu-client/src/main/java/org/apache/kudu/client/TableLocationsCache.java M java/kudu-client/src/main/java/org/apache/kudu/client/TabletClient.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduSession.java 8 files changed, 345 insertions(+), 318 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4719 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d06a573e4307c76a7aebab05cd5238fb0d9a2c6 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins
