You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by le...@apache.org on 2019/03/11 16:52:53 UTC
[incubator-druid] branch master updated: Remove unnecessary check
for contains() in LoadRule (#7073)
This is an automated email from the ASF dual-hosted git repository.
leventov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new 8804bd0 Remove unnecessary check for contains() in LoadRule (#7073)
8804bd0 is described below
commit 8804bd0dc10c4b0d3f63179501c96259b91d798e
Author: Samarth Jain <sa...@apache.org>
AuthorDate: Mon Mar 11 09:52:46 2019 -0700
Remove unnecessary check for contains() in LoadRule (#7073)
See https://github.com/apache/incubator-druid/issues/7072
---
.../java/org/apache/druid/server/coordinator/rules/LoadRule.java | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/server/src/main/java/org/apache/druid/server/coordinator/rules/LoadRule.java b/server/src/main/java/org/apache/druid/server/coordinator/rules/LoadRule.java
index 1de3479..3b2b642 100644
--- a/server/src/main/java/org/apache/druid/server/coordinator/rules/LoadRule.java
+++ b/server/src/main/java/org/apache/druid/server/coordinator/rules/LoadRule.java
@@ -74,10 +74,7 @@ public abstract class LoadRule implements Rule
currentReplicants.putAll(params.getSegmentReplicantLookup().getClusterTiers(segment.getId()));
final CoordinatorStats stats = new CoordinatorStats();
-
- if (params.getAvailableSegments().contains(segment)) {
- assign(params, segment, stats);
- }
+ assign(params, segment, stats);
drop(params, segment, stats);
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org