Todd Lipcon has posted comments on this change. Change subject: c++ client: use operation timeout as deadline for finding new leader master ......................................................................
Patch Set 2: to work around in that particular test, why not explicitly wait for a leader election on the masters? I think the user experience is not so great to say that if your cluster is down you have to wait 60 seconds to get an error (even though you may be willing to wait 60 seconds if you seem to be making some progress). Perhaps we can "early out" in the case that you get NetworkError from _all_ of the potential masters? -- To view, visit http://gerrit.cloudera.org:8080/3718 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0d770875bbf4703444abac11dbc232d7e382165e Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: No