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 <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: No

Reply via email to