Jean-Daniel Cryans has submitted this change and it was merged. Change subject: c++ client: use entire deadline when retrying single-master DDL operations ......................................................................
c++ client: use entire deadline when retrying single-master DDL operations I was perplexed when I saw a new test that uses TestWorkload timeout in CreateTable() after only 10s when the default admin operation timeout is 30s. Turns out, it's an actual bug in the client. Of course, if CreateTable() is retried after 10s of waiting, it'll get either ServiceUnavailable() or AlreadyPresent() until KUDU-1537 is fixed. Change-Id: I23e6312bc014d59da04a0653d85d8ab1612d78d1 Reviewed-on: http://gerrit.cloudera.org:8080/4435 Tested-by: Adar Dembo <a...@cloudera.com> Reviewed-by: Jean-Daniel Cryans <jdcry...@apache.org> --- M src/kudu/client/client-internal.cc M src/kudu/integration-tests/test_workload.cc 2 files changed, 11 insertions(+), 7 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Adar Dembo: Verified -- To view, visit http://gerrit.cloudera.org:8080/4435 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23e6312bc014d59da04a0653d85d8ab1612d78d1 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Tidy Bot