Skip to content

Commit 8fe9637

Browse files
committed
Removed unneeded ParsedAggregations class
1 parent f928035 commit 8fe9637

File tree

4 files changed

+3
-64
lines changed

4 files changed

+3
-64
lines changed

core/src/main/java/org/elasticsearch/search/aggregations/Aggregations.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ public class Aggregations implements Iterable<Aggregation>, ToXContent {
4747
protected Aggregations() {
4848
}
4949

50-
protected Aggregations(List<? extends Aggregation> aggregations) {
50+
public Aggregations(List<? extends Aggregation> aggregations) {
5151
this.aggregations = aggregations;
5252
}
5353

core/src/main/java/org/elasticsearch/search/aggregations/ParsedAggregations.java

Lines changed: 0 additions & 57 deletions
This file was deleted.

core/src/main/java/org/elasticsearch/search/aggregations/bucket/ParsedSingleBucketAggregation.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
import org.elasticsearch.search.aggregations.Aggregation;
2626
import org.elasticsearch.search.aggregations.Aggregations;
2727
import org.elasticsearch.search.aggregations.ParsedAggregation;
28-
import org.elasticsearch.search.aggregations.ParsedAggregations;
2928

3029
import java.io.IOException;
3130
import java.util.ArrayList;
@@ -38,7 +37,7 @@ public abstract class ParsedSingleBucketAggregation extends ParsedAggregation im
3837

3938
private long docCount;
4039
protected List<Aggregation> aggregationList = new ArrayList<>();
41-
private ParsedAggregations aggregations = new ParsedAggregations(aggregationList);
40+
private Aggregations aggregations = new Aggregations(aggregationList);
4241

4342
@Override
4443
public long getDocCount() {

core/src/test/java/org/elasticsearch/search/aggregations/bucket/InternalSingleBucketAggregationTestCase.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,5 @@ protected void assertFromXContent(T aggregation, ParsedAggregation parsedAggrega
109109
assertTrue(parsedClass != null && parsedClass.isInstance(parsedAggregation));
110110
}
111111

112-
// TODO make abstract
113-
protected Class<? extends ParsedSingleBucketAggregation> implementationClass() {
114-
return ParsedSingleBucketAggregation.class;
115-
}
112+
protected abstract Class<? extends ParsedSingleBucketAggregation> implementationClass();
116113
}

0 commit comments

Comments
 (0)