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

----


---

Reply via email to