Hello Jean-Daniel Cryans,

I'd like you to do a code review.  Please visit

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

to review the following change.

Change subject: WIP: java: use new ConnectToMaster RPC with fallback
......................................................................

WIP: java: use new ConnectToMaster RPC with fallback

This changes the Java client from using GetMasterRegistration to using
ConnectToMaster. If the call fails with an error that the method doesn't
exist, it falls back to GetMasterRegistration.

This is the first step towards grabbing authentication tokens and TLS
certificates from the master when connecting to a cluster.

WIP: should add a test for the fallback path, though I did test it
manually.

Change-Id: Idcb7c5e56a53c788cf96468f235db73b46a3d629
---
M java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToCluster.java
M 
java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToClusterResponse.java
M 
java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToMasterRequest.java
A java/kudu-client/src/main/java/org/apache/kudu/client/RpcRemoteException.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/TestConnectToCluster.java
6 files changed, 131 insertions(+), 44 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idcb7c5e56a53c788cf96468f235db73b46a3d629
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>

Reply via email to