git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[incubator-druid] branch 0.12.2 updated: coordinator slight optimze load rule to skip drop if numToDrop is 0 (#5928) (#5986)


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

jihoonson pushed a commit to branch 0.12.2
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git


The following commit(s) were added to refs/heads/0.12.2 by this push:
     new a34d620  coordinator slight optimze load rule to skip drop if numToDrop is 0 (#5928) (#5986)
a34d620 is described below

commit a34d620de6a9508f15a7b7d8a8187693448613bd
Author: Jihoon Son <jihoonson@xxxxxxxxxx>
AuthorDate: Mon Jul 9 14:07:22 2018 -0700

    coordinator slight optimze load rule to skip drop if numToDrop is 0 (#5928) (#5986)
---
 .../src/main/java/io/druid/server/coordinator/rules/LoadRule.java   | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/server/src/main/java/io/druid/server/coordinator/rules/LoadRule.java b/server/src/main/java/io/druid/server/coordinator/rules/LoadRule.java
index 094dedd..b8ea703 100644
--- a/server/src/main/java/io/druid/server/coordinator/rules/LoadRule.java
+++ b/server/src/main/java/io/druid/server/coordinator/rules/LoadRule.java
@@ -322,7 +322,11 @@ public abstract class LoadRule implements Rule
       } else {
         final int currentReplicantsInTier = entry.getIntValue();
         final int numToDrop = currentReplicantsInTier - targetReplicants.getOrDefault(tier, 0);
-        numDropped = dropForTier(numToDrop, holders, segment, params.getBalancerStrategy());
+        if (numToDrop > 0) {
+          numDropped = dropForTier(numToDrop, holders, segment, params.getBalancerStrategy());
+        } else {
+          numDropped = 0;
+        }
       }
 
       stats.addToTieredStat(DROPPED_COUNT, tier, numDropped);


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@xxxxxxxxxxxxxxxx
For additional commands, e-mail: dev-help@xxxxxxxxxxxxxxxx