Todd Lipcon has submitted this change and it was merged.

Change subject: maintenance_manager: simplify test
......................................................................


maintenance_manager: simplify test

This simplifies maintenance_manager-test a bit to avoid a condition
variable, and instead just poll from the main thread. This reduces the
complexity to understand the test a bit, and the test still runs quickly
(<100ms).

This also changes the test operation in preparation for a later patch
which will allow an operation to run multiple times in parallel and
auto-disable itself after a prescribed number of runs.

Change-Id: I265318f64771803714f48a6e96fd3260b6eafae4
Reviewed-on: http://gerrit.cloudera.org:8080/4294
Reviewed-by: Adar Dembo <a...@cloudera.com>
Tested-by: Kudu Jenkins
---
M src/kudu/util/maintenance_manager-test.cc
1 file changed, 38 insertions(+), 60 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I265318f64771803714f48a6e96fd3260b6eafae4
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to