Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11107 )

Change subject: [tools] run rebalancer during 'election storm'
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11107/1/src/kudu/tools/kudu-admin-test.cc
File src/kudu/tools/kudu-admin-test.cc:

http://gerrit.cloudera.org:8080/#/c/11107/1/src/kudu/tools/kudu-admin-test.cc@2093
PS1, Line 2093: auto max_sleep_ms = 2.0;
> What do you mean by "module expression"?
Oh, that was a typo.  I meant 'modulo' :)

And then there is std::min(x * 1.1, 2000.0) step.

I added corresponding comment into the code.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic98684dbe55049bbc411513faa0b6bbaef20f434
Gerrit-Change-Number: 11107
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Thu, 02 Aug 2018 21:50:13 +0000
Gerrit-HasComments: Yes

Reply via email to