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

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
Reviewed-on: http://gerrit.cloudera.org:8080/11718
Tested-by: Andrew Wong <[email protected]>
Reviewed-by: Adar Dembo <[email protected]>
---
M src/kudu/tserver/tablet_server-test-base.cc
1 file changed, 15 insertions(+), 2 deletions(-)

Approvals:
  Andrew Wong: Verified
  Adar Dembo: Looks good to me, approved

--
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: merged
Gerrit-Change-Id: Ie6ca079353d4d44a48defc386d94b4bda452ec1b
Gerrit-Change-Number: 11718
Gerrit-PatchSet: 3
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Tidy Bot (241)

Reply via email to