Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15224
to look at the new patch set (#2).
Change subject: [test] fix flake in TsTabletManagerITest::TestTableStats
......................................................................
[test] fix flake in TsTabletManagerITest::TestTableStats
In rare cases, leader replica of a tablet can change its role prior to
receiving StepDown request, so the test outputs somelike like:
I0214 18:13:07.095886 5028 raft_consensus.cc:572] T
a32ee1063340424db8a7e7ea6aedcfa1 P ddfa34be67fd4a40b26218185403580c [term 3
FOLLOWER]: Rejecting request to step down while not leader
src/kudu/integration-tests/ts_tablet_manager-itest.cc:818: Failure
Failed
Bad status: Illegal state: Code NOT_THE_LEADER: Not currently leader
To avoid test flakiness it makes sense to continue with next tablet
when observing such an error.
To verify the fix, I ran the scenario in TSAN configuration before
with and without this patch:
before: 16 failures in 256 runs
after: 0 failures in 256 runs
Note, however, that this scenario is still incredibly flaky, but
at least one source of flakiness is gone.
Change-Id: Ibbd6652cdcb30f8899767dfb932cc402cf739115
---
M src/kudu/integration-tests/ts_tablet_manager-itest.cc
1 file changed, 12 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/24/15224/2
--
To view, visit http://gerrit.cloudera.org:8080/15224
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ibbd6652cdcb30f8899767dfb932cc402cf739115
Gerrit-Change-Number: 15224
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
