Skip to content

Commit fe7fd09

Browse files
author
Paul Sanwald
committed
Revert "Adds a new auto-interval date histogram (elastic#28993)"
This reverts commit d88d764.
1 parent 9744990 commit fe7fd09

20 files changed

+6
-3263
lines changed

client/rest-high-level/src/main/java/org/elasticsearch/client/RestHighLevelClient.java

-3
Original file line numberDiff line numberDiff line change
@@ -85,10 +85,8 @@
8585
import org.elasticsearch.search.aggregations.bucket.geogrid.ParsedGeoHashGrid;
8686
import org.elasticsearch.search.aggregations.bucket.global.GlobalAggregationBuilder;
8787
import org.elasticsearch.search.aggregations.bucket.global.ParsedGlobal;
88-
import org.elasticsearch.search.aggregations.bucket.histogram.AutoDateHistogramAggregationBuilder;
8988
import org.elasticsearch.search.aggregations.bucket.histogram.DateHistogramAggregationBuilder;
9089
import org.elasticsearch.search.aggregations.bucket.histogram.HistogramAggregationBuilder;
91-
import org.elasticsearch.search.aggregations.bucket.histogram.ParsedAutoDateHistogram;
9290
import org.elasticsearch.search.aggregations.bucket.histogram.ParsedDateHistogram;
9391
import org.elasticsearch.search.aggregations.bucket.histogram.ParsedHistogram;
9492
import org.elasticsearch.search.aggregations.bucket.missing.MissingAggregationBuilder;
@@ -1349,7 +1347,6 @@ static List<NamedXContentRegistry.Entry> getDefaultNamedXContents() {
13491347
map.put(GeoCentroidAggregationBuilder.NAME, (p, c) -> ParsedGeoCentroid.fromXContent(p, (String) c));
13501348
map.put(HistogramAggregationBuilder.NAME, (p, c) -> ParsedHistogram.fromXContent(p, (String) c));
13511349
map.put(DateHistogramAggregationBuilder.NAME, (p, c) -> ParsedDateHistogram.fromXContent(p, (String) c));
1352-
map.put(AutoDateHistogramAggregationBuilder.NAME, (p, c) -> ParsedAutoDateHistogram.fromXContent(p, (String) c));
13531350
map.put(StringTerms.NAME, (p, c) -> ParsedStringTerms.fromXContent(p, (String) c));
13541351
map.put(LongTerms.NAME, (p, c) -> ParsedLongTerms.fromXContent(p, (String) c));
13551352
map.put(DoubleTerms.NAME, (p, c) -> ParsedDoubleTerms.fromXContent(p, (String) c));

docs/reference/aggregations/bucket.asciidoc

-2
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,6 @@ setting named `search.max_buckets`. It is disabled by default (-1) but requests
1919

2020
include::bucket/adjacency-matrix-aggregation.asciidoc[]
2121

22-
include::bucket/autodatehistogram-aggregation.asciidoc[]
23-
2422
include::bucket/children-aggregation.asciidoc[]
2523

2624
include::bucket/composite-aggregation.asciidoc[]

docs/reference/aggregations/bucket/autodatehistogram-aggregation.asciidoc

-283
This file was deleted.

server/src/main/java/org/elasticsearch/search/SearchModule.java

-4
Original file line numberDiff line numberDiff line change
@@ -109,10 +109,8 @@
109109
import org.elasticsearch.search.aggregations.bucket.geogrid.InternalGeoHashGrid;
110110
import org.elasticsearch.search.aggregations.bucket.global.GlobalAggregationBuilder;
111111
import org.elasticsearch.search.aggregations.bucket.global.InternalGlobal;
112-
import org.elasticsearch.search.aggregations.bucket.histogram.AutoDateHistogramAggregationBuilder;
113112
import org.elasticsearch.search.aggregations.bucket.histogram.DateHistogramAggregationBuilder;
114113
import org.elasticsearch.search.aggregations.bucket.histogram.HistogramAggregationBuilder;
115-
import org.elasticsearch.search.aggregations.bucket.histogram.InternalAutoDateHistogram;
116114
import org.elasticsearch.search.aggregations.bucket.histogram.InternalDateHistogram;
117115
import org.elasticsearch.search.aggregations.bucket.histogram.InternalHistogram;
118116
import org.elasticsearch.search.aggregations.bucket.missing.InternalMissing;
@@ -398,8 +396,6 @@ private void registerAggregations(List<SearchPlugin> plugins) {
398396
HistogramAggregationBuilder::parse).addResultReader(InternalHistogram::new));
399397
registerAggregation(new AggregationSpec(DateHistogramAggregationBuilder.NAME, DateHistogramAggregationBuilder::new,
400398
DateHistogramAggregationBuilder::parse).addResultReader(InternalDateHistogram::new));
401-
registerAggregation(new AggregationSpec(AutoDateHistogramAggregationBuilder.NAME, AutoDateHistogramAggregationBuilder::new,
402-
AutoDateHistogramAggregationBuilder::parse).addResultReader(InternalAutoDateHistogram::new));
403399
registerAggregation(new AggregationSpec(GeoDistanceAggregationBuilder.NAME, GeoDistanceAggregationBuilder::new,
404400
GeoDistanceAggregationBuilder::parse).addResultReader(InternalGeoDistance::new));
405401
registerAggregation(new AggregationSpec(GeoGridAggregationBuilder.NAME, GeoGridAggregationBuilder::new,

server/src/main/java/org/elasticsearch/search/aggregations/bucket/BucketsAggregator.java

-13
Original file line numberDiff line numberDiff line change
@@ -84,19 +84,6 @@ public final void collectExistingBucket(LeafBucketCollector subCollector, int do
8484
subCollector.collect(doc, bucketOrd);
8585
}
8686

87-
public final void mergeBuckets(long[] mergeMap, long newNumBuckets) {
88-
try (IntArray oldDocCounts = docCounts) {
89-
docCounts = bigArrays.newIntArray(newNumBuckets, true);
90-
docCounts.fill(0, newNumBuckets, 0);
91-
for (int i = 0; i < oldDocCounts.size(); i++) {
92-
int docCount = oldDocCounts.get(i);
93-
if (docCount != 0) {
94-
docCounts.increment(mergeMap[i], docCount);
95-
}
96-
}
97-
}
98-
}
99-
10087
public IntArray getDocCounts() {
10188
return docCounts;
10289
}

0 commit comments

Comments
 (0)