Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/4435

to look at the new patch set (#2).

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
---
M src/kudu/client/client-internal.cc
M src/kudu/integration-tests/test_workload.cc
2 files changed, 11 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/35/4435/2
-- 
To view, visit http://gerrit.cloudera.org:8080/4435
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I23e6312bc014d59da04a0653d85d8ab1612d78d1
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot

Reply via email to