GitHub user narendly opened a pull request:

    https://github.com/apache/helix/pull/245

    Fix TestStateTransitionThrottle and comparison operator change

    …hange
    
    Due to the change made in relation to allowing downward state transitions 
to take place while error or recovery balance transitions are present, this 
test was failing due to the change in the assumption. Parameters were adjusted, 
and test conditions were modified such that it is testing the new assumptions 
correctly.
    
    Changelist:
    1. TestStateTransitionThrottle assert statements were modified so that it 
assumes downward load balance transitions taking place
    2. Comparison operator in IntermediateStateCalcState to make it more 
strictly backward-compatible

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/narendly/helix 1340493

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/helix/pull/245.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 #245
    
----
commit 562115e5bedcc1c8d4d3e5b67962cb03e7a37bf9
Author: Hunter Lee <narendly@...>
Date:   2018-07-16T18:48:19Z

    [HELIX-731] Fix TestStateTransitionThrottle and comparison operator change
    
    Due to the change made in relation to allowing downward state transitions 
to take place while error or recovery balance transitions are present, this 
test was failing due to the change in the assumption. Parameters were adjusted, 
and test conditions were modified such that it is testing the new assumptions 
correctly.
    
    Changelist:
    1. TestStateTransitionThrottle assert statements were modified so that it 
assumes downward load balance transitions taking place
    2. Comparison operator in IntermediateStateCalcState to make it more 
strictly backward-compatible

----


---

Reply via email to