Hello Will Berkeley, Mike Percy, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10920
to look at the new patch set (#3).
Change subject: [tools] sanity check for ScheduleReplicaMove()
......................................................................
[tools] sanity check for ScheduleReplicaMove()
This change introduces a couple of improvements into the implementation
of the kudu::tools::ScheduleReplicaMove() function, used by the
kudu CLI tool's 'kudu tablet change_config move_replica' and
'kudu cluster rebalance' sub-commands:
* use CAS semantics when changing tablet Raft configuration
* avoid setting the REPLACE attribute if it's set already
The former protects against unexpected Raft configuration changes
in the middle of the replica movement process. The latter helps in
situations when a configuration change initiated by a prior run of
the rebalancing tool failed at later stages -- with this change,
there is no need to reset the source replica attributes separately.
This changelist does not add corresponding tests since the existing
ConcurrentRebalancersTest.TwoConcurrentRebalancers scenario provides
enough coverage, IMO. As a result of this change, the test scenario
became more stable when running with --stress_cpu_threads=16 flag.
before (24 out of 256 failed):
http://dist-test.cloudera.org/job?job_id=aserbin.1531287387.25723
after (none of 256 failed):
http://dist-test.cloudera.org/job?job_id=aserbin.1531288786.42701
Change-Id: Ie311b4bb2dbe3e5f1e86cb1364039b71d7c08019
---
M src/kudu/tools/kudu-admin-test.cc
M src/kudu/tools/tool_replica_util.cc
2 files changed, 26 insertions(+), 11 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/20/10920/3
--
To view, visit http://gerrit.cloudera.org:8080/10920
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie311b4bb2dbe3e5f1e86cb1364039b71d7c08019
Gerrit-Change-Number: 10920
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
