Skip to content

Commit 1dacb70

Browse files
author
Luc Boutier
committed
Remove useless filters on aggregations.
1 parent 12d9f4f commit 1dacb70

File tree

1 file changed

+4
-12
lines changed

1 file changed

+4
-12
lines changed

elasticsearch-mapping/src/main/java/org/elasticsearch/mapping/QueryHelper.java

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -488,26 +488,18 @@ private void addFilters(SearchRequestBuilder searchRequestBuilder, FilterBuilder
488488
}
489489
if (facets) {
490490
if (filters == null) {
491-
addAggregations(new HashMap<String, String[]>(), clazz.getName(), searchRequestBuilder, filter);
491+
addAggregations(new HashMap<String, String[]>(), clazz.getName(), searchRequestBuilder);
492492
} else {
493-
addAggregations(filters, clazz.getName(), searchRequestBuilder, filter);
493+
addAggregations(filters, clazz.getName(), searchRequestBuilder);
494494
}
495495
}
496496
}
497497

498-
private void addAggregations(Map<String, String[]> filters, String className, SearchRequestBuilder searchRequestBuilder, FilterBuilder filter) {
498+
private void addAggregations(Map<String, String[]> filters, String className, SearchRequestBuilder searchRequestBuilder) {
499499
final List<AggregationBuilder> aggregations = buildAggregations(className, filters.keySet());
500500

501501
if (aggregations.size() > 0) {
502-
AggregationBuilder aggregationBuilder;
503-
504-
if (filter == null) {
505-
// In order to gather all unfiltered aggregations faceted results under one single parent aggregation, a Global Aggregation is used
506-
aggregationBuilder = AggregationBuilders.global("global_aggregation");
507-
} else {
508-
// To include filters inside filtered aggregation results
509-
aggregationBuilder = AggregationBuilders.filters("filter_aggregation").filter(filter);
510-
}
502+
AggregationBuilder aggregationBuilder = AggregationBuilders.global("facet_aggregation");
511503

512504
for (AggregationBuilder aggregation : aggregations) {
513505
aggregationBuilder.subAggregation(aggregation);

0 commit comments

Comments
 (0)