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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
