Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15025 )

Change subject: maintenance_manager-test: deflake TestPriorityOpLaunch
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15025/1/src/kudu/util/maintenance_manager-test.cc
File src/kudu/util/maintenance_manager-test.cc:

http://gerrit.cloudera.org:8080/#/c/15025/1/src/kudu/util/maintenance_manager-test.cc@539
PS1, Line 539:   TestMaintenanceOp op5("op5", MaintenanceOp::HIGH_IO_USAGE, 
FLAGS_max_priority_range + 1);
             :   op5.set_perf_improvement(10);
             :   op5.set_remaining_runs(1);
             :   op5.set_sleep_time(MonoDelta::FromMilliseconds(1));
             :
             :   TestMaintenanceOp op6("op6", MaintenanceOp::HIGH_IO_USAGE, 0);
             :   op6.set_perf_improvement(12);
             :   op6.set_remaining_runs(1);
             :   op6.set_sleep_time(MonoDelta::FromMilliseconds(1));
> nit: while you're here, would you mind flipping the names here and reversin
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I06f3fd8fa952ed74bfaf2ac3dd55cd289f4b878b
Gerrit-Change-Number: 15025
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 13 Jan 2020 23:03:20 +0000
Gerrit-HasComments: Yes

Reply via email to