Todd Lipcon 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.
> Also I just double checked, and ConnectToMasterResponsePB doesn't include t
Example alternate patch which also seems to fix the issue:
https://gist.github.com/8f0f8e38b04fb567f014e248e6f5d79b

(maybe this is better for a 1.7-targeted and backportable fix?)



--
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-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Comment-Date: Tue, 13 Mar 2018 21:15:26 +0000
Gerrit-HasComments: Yes

Reply via email to