Andrew Wong has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16463 )

Change subject: test: deflake tablet_server_quiescing-itest
......................................................................

test: deflake tablet_server_quiescing-itest

It was possible that TestScansRetry would fail with more than the one
active scanner at the end of the test. This is possible because the
client scanner in the test that may have lingering retries from a
previous scan attmpts.

This patch updates to allow for multiple scan attempts, since it doesn't
detract from testing the fact that the keep-alive calls are working.

It also reenstates the quiescing state before calling the keep-alives,
which appears to be the original intent based on comments.

Change-Id: Ib6525b913cbe9acbdbc59d8a52df81c85327d0c7
Reviewed-on: http://gerrit.cloudera.org:8080/16463
Reviewed-by: Grant Henke <granthe...@apache.org>
Tested-by: Kudu Jenkins
---
M src/kudu/integration-tests/tablet_server_quiescing-itest.cc
1 file changed, 8 insertions(+), 3 deletions(-)

Approvals:
  Grant Henke: Looks good to me, approved
  Kudu Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/16463
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib6525b913cbe9acbdbc59d8a52df81c85327d0c7
Gerrit-Change-Number: 16463
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to