You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@helix.apache.org by Kanak Biscuitwala <ka...@hotmail.com> on 2013/09/04 00:50:16 UTC
Review Request 13951: [HELIX-213] ZkHelixAdmin#rebalance() should not assume
master-slave state model
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13951/
-----------------------------------------------------------
Review request for helix, Zhen Zhang, Kishore Gopalakrishna, and Shi Lu.
Bugs: HELIX-213
Repository: helix-git
Description
-------
commit 33036e11a90ebc93add8d280bc9ad2d7f839c56a
Author: Kanak Biscuitwala <ka...@hotmail.com>
Date: Tue Sep 3 15:43:42 2013 -0700
[HELIX-213] ZkHelixAdmin#rebalance() should not assume master-slave state model
:100644 100644 9564e35... 99df686... M helix-core/src/main/java/org/apache/helix/controller/rebalancer/AutoRebalancer.java
:100644 100644 b096817... 70dba89... M helix-core/src/main/java/org/apache/helix/controller/rebalancer/SemiAutoRebalancer.java
:100644 100644 d2dbdef... 11518a5... M helix-core/src/main/java/org/apache/helix/controller/rebalancer/util/ConstraintBasedAssignment.java
:100644 100644 72046bf... e931163... M helix-core/src/main/java/org/apache/helix/controller/strategy/AutoRebalanceStrategy.java
:100644 100644 754df7b... e04a890... M helix-core/src/main/java/org/apache/helix/manager/zk/ZKHelixAdmin.java
:100644 100644 9fb7ba9... e357b55... M helix-core/src/test/java/org/apache/helix/controller/strategy/TestAutoRebalanceStrategy.java
Diffs
-----
helix-core/src/main/java/org/apache/helix/controller/rebalancer/AutoRebalancer.java 9564e35
helix-core/src/main/java/org/apache/helix/controller/rebalancer/SemiAutoRebalancer.java b096817
helix-core/src/main/java/org/apache/helix/controller/rebalancer/util/ConstraintBasedAssignment.java d2dbdef
helix-core/src/main/java/org/apache/helix/controller/strategy/AutoRebalanceStrategy.java 72046bf
helix-core/src/main/java/org/apache/helix/manager/zk/ZKHelixAdmin.java 754df7b
helix-core/src/test/java/org/apache/helix/controller/strategy/TestAutoRebalanceStrategy.java 9fb7ba9
Diff: https://reviews.apache.org/r/13951/diff/
Testing
-------
Tests pass locally.
Thanks,
Kanak Biscuitwala