[helix] branch master updated: IllegalStateException for CRUSH based rebalance strategy algorithm.

[jxue]

This is an automated email from the ASF dual-hosted git repository.

jxue pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/helix.git


The following commit(s) were added to refs/heads/master by this push:
     new c0dc0e5  IllegalStateException for CRUSH based rebalance strategy 
algorithm.
c0dc0e5 is described below

commit c0dc0e5163586687e53d01fbb480570b165a20b1
Author: Ali Najari <anaj...@anajari-ld1.linkedin.biz>
AuthorDate: Mon Jul 15 15:25:52 2019 -0700

    IllegalStateException for CRUSH based rebalance strategy algorithm.
    
    This commit fixes the error log and exception that is shown when there is 
not enough eligible instance to use.
---
 .../org/apache/helix/controller/stages/BestPossibleStateCalcStage.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/helix-core/src/main/java/org/apache/helix/controller/stages/BestPossibleStateCalcStage.java
 
b/helix-core/src/main/java/org/apache/helix/controller/stages/BestPossibleStateCalcStage.java
index b508796..9bbf12a 100644
--- 
a/helix-core/src/main/java/org/apache/helix/controller/stages/BestPossibleStateCalcStage.java
+++ 
b/helix-core/src/main/java/org/apache/helix/controller/stages/BestPossibleStateCalcStage.java
@@ -259,7 +259,7 @@ public class BestPossibleStateCalcStage extends 
AbstractBaseStage {
         return checkBestPossibleStateCalculation(idealState);
       } catch (Exception e) {
         LogUtil
-            .logError(logger, _eventId, "Error computing assignment for 
resource " + resourceName + ". Skipping.", e);
+            .logError(logger, _eventId, "Error computing assignment for 
resource " + resourceName + ". Skipping.");
         // TODO : remove this part after debugging NPE
         StringBuilder sb = new StringBuilder();