GitHub user juvenzhu opened a pull request:
https://github.com/apache/helix/pull/136
[helix-core] _rebalanceTimerPeriod in RebalanceConfig should be in
milliseconds
This left us some confusion since REBALANCE_TIMER_PERIOD in
ResourceConfigProperty is not actually used.
However, REBALANCE_TIMER_PERIOD in IdealStateProperty &
ClusterConfigProperty are used in GenericHelixController.java and they are in
milliseconds.
Maybe there were some refactoring going on but set this in seconds would
leave pp some confusions.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/juvenzhu/helix master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/helix/pull/136.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #136
----
commit 8aa2abe10ed830aeb5969335922fb4c74e0e61df
Author: Weihua Zhu <wei@...>
Date: 2018-02-23T07:32:21Z
change seconds to milliseconds
----
---
