Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9612 )

Change subject: KUDU-2343. java: properly reconnect to new leader master after 
failover
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/9612/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/9612/1//COMMIT_MSG@10
PS1, Line 10: We don't currently use the
            : master UUIDs in the Java client, so the connections to all 
masters were
            : getting conflated in the cache.
Just passing through, but can you show me where this happens? I dug around and 
found AsyncKuduClient.newMasterRpcProxy, which creates a ServerInfo with UUID 
"master-<hostport>" and that ought to uniquely identify every master in a 
multi-master cluster.



--
To view, visit http://gerrit.cloudera.org:8080/9612
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I36f96c6712800e398ed46887d97d4b09fd993b04
Gerrit-Change-Number: 9612
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward #314
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Tue, 13 Mar 2018 20:49:02 +0000
Gerrit-HasComments: Yes

Reply via email to