From 68e8189b95891b1fe5d2dc6039475f13199aa366 Mon Sep 17 00:00:00 2001 From: iverase Date: Tue, 19 Sep 2023 11:27:23 +0200 Subject: [PATCH] Remove supportsParallelCollection implementation for some aggs --- .../bucket/composite/CompositeAggregationBuilder.java | 5 ----- .../histogram/VariableWidthHistogramAggregationBuilder.java | 5 ----- .../aggregations/bucket/nested/NestedAggregationBuilder.java | 5 ----- .../aggregations/metrics/CardinalityAggregationBuilder.java | 5 ----- .../frequentitemsets/FrequentItemSetsAggregationBuilder.java | 5 ----- 5 files changed, 25 deletions(-) diff --git a/server/src/main/java/org/elasticsearch/search/aggregations/bucket/composite/CompositeAggregationBuilder.java b/server/src/main/java/org/elasticsearch/search/aggregations/bucket/composite/CompositeAggregationBuilder.java index 6b894b3a9dbb..ca88d5089876 100644 --- a/server/src/main/java/org/elasticsearch/search/aggregations/bucket/composite/CompositeAggregationBuilder.java +++ b/server/src/main/java/org/elasticsearch/search/aggregations/bucket/composite/CompositeAggregationBuilder.java @@ -97,11 +97,6 @@ public boolean supportsSampling() { return true; } - @Override - public boolean supportsParallelCollection() { - return false; - } - public CompositeAggregationBuilder(StreamInput in) throws IOException { super(in); int num = in.readVInt(); diff --git a/server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/VariableWidthHistogramAggregationBuilder.java b/server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/VariableWidthHistogramAggregationBuilder.java index 9ca606f460fb..f7870df45648 100644 --- a/server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/VariableWidthHistogramAggregationBuilder.java +++ b/server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/VariableWidthHistogramAggregationBuilder.java @@ -137,11 +137,6 @@ public BucketCardinality bucketCardinality() { return BucketCardinality.MANY; } - @Override - public boolean supportsParallelCollection() { - return false; - } - @Override protected AggregationBuilder shallowCopy(AggregatorFactories.Builder factoriesBuilder, Map metaData) { return new VariableWidthHistogramAggregationBuilder(this, factoriesBuilder, metaData); diff --git a/server/src/main/java/org/elasticsearch/search/aggregations/bucket/nested/NestedAggregationBuilder.java b/server/src/main/java/org/elasticsearch/search/aggregations/bucket/nested/NestedAggregationBuilder.java index b6e6a39356d4..f4ba3db38358 100644 --- a/server/src/main/java/org/elasticsearch/search/aggregations/bucket/nested/NestedAggregationBuilder.java +++ b/server/src/main/java/org/elasticsearch/search/aggregations/bucket/nested/NestedAggregationBuilder.java @@ -81,11 +81,6 @@ public BucketCardinality bucketCardinality() { return BucketCardinality.ONE; } - @Override - public boolean supportsParallelCollection() { - return false; - } - @Override protected AggregatorFactory doBuild(AggregationContext context, AggregatorFactory parent, Builder subFactoriesBuilder) throws IOException { diff --git a/server/src/main/java/org/elasticsearch/search/aggregations/metrics/CardinalityAggregationBuilder.java b/server/src/main/java/org/elasticsearch/search/aggregations/metrics/CardinalityAggregationBuilder.java index 03e3dc93675d..615b46434226 100644 --- a/server/src/main/java/org/elasticsearch/search/aggregations/metrics/CardinalityAggregationBuilder.java +++ b/server/src/main/java/org/elasticsearch/search/aggregations/metrics/CardinalityAggregationBuilder.java @@ -113,11 +113,6 @@ public boolean supportsSampling() { return true; } - @Override - public boolean supportsParallelCollection() { - return false; - } - @Override protected boolean serializeTargetValueType(TransportVersion version) { return true; diff --git a/x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/frequentitemsets/FrequentItemSetsAggregationBuilder.java b/x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/frequentitemsets/FrequentItemSetsAggregationBuilder.java index 9081f4118f89..684d61dbdedb 100644 --- a/x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/frequentitemsets/FrequentItemSetsAggregationBuilder.java +++ b/x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/frequentitemsets/FrequentItemSetsAggregationBuilder.java @@ -182,11 +182,6 @@ public boolean supportsSampling() { return true; } - @Override - public boolean supportsParallelCollection() { - return false; - } - @Override protected AggregationBuilder shallowCopy(Builder factoriesBuilder, Map metadata) { return new FrequentItemSetsAggregationBuilder(name, fields, minimumSupport, minimumSetSize, size, filter, executionHint);