Alexey Serbin has posted comments on this change. Change subject: [c++ client] use default RPC timeout for batchers ......................................................................
Patch Set 1: > > Can we add a new unit test of sorts that pauses a leader and > makes > > sure that a write to that tablet succeeds? I think it used to not > > succeed because it used the full operation deadline on that > write. > > Good idea. Do you prefer that to be a part of this change or make > that in a separate changelist? OK, I'll address it in a separate change list. -- To view, visit http://gerrit.cloudera.org:8080/4703 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic44acd5e2723295e115677f83e3fcf4074a61e52 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: No
