junkaixue commented on a change in pull request #1750:
URL: https://github.com/apache/helix/pull/1750#discussion_r638378537
##########
File path:
helix-core/src/test/java/org/apache/helix/controller/stages/TestRecoveryLoadBalance.java
##########
@@ -57,7 +57,10 @@
private final String STATE_MODEL = "statemodel";
private ClusterConfig _clusterConfig;
- @Test(dataProvider = "recoveryLoadBalanceInput")
+ // Disable the test since recovery rebalance is not able to block load
rebalance.
+ // TODO: if we decide to support that by replica level, we will have a
different config for that and this test can be
Review comment:
I will have a new test class already covered most of cases.
Disable the test is keep the possibility of supporting the feature again. As
I said, this feature is not determined to implement or not.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
