Hello Tidy Bot, Mike Percy, Kudu Jenkins, Adar Dembo,

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

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

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

Change subject: tablet_server-test: wait for leader in some tests
......................................................................

tablet_server-test: wait for leader in some tests

The fix for KUDU-2463 made successful Raft elections a prerequisite for
scanning empty tablets. Under enough stress, even in a single replica
case, this may not complete immediately, and can thus, lead scans to
return a different error than expected. This patch updates the test
fixture that creates an empty tablet in tablet_server-test to wait until
this has passed before deeming the tablet ready for business.

The following test incantation:
tablet_server-test --gtest_filter=*BadProjectionTypes* --stress_cpu_threads=100

would fail fairly consistently without this patch.

Change-Id: Ie6ca079353d4d44a48defc386d94b4bda452ec1b
---
M src/kudu/tserver/tablet_server-test-base.cc
1 file changed, 15 insertions(+), 2 deletions(-)


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie6ca079353d4d44a48defc386d94b4bda452ec1b
Gerrit-Change-Number: 11718
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Tidy Bot

Reply via email to