honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Don't "double" cache a facet filter / query facet, closes #734.

This commit is contained in:
kimchy 2011-03-01 22:14:36 +02:00
parent 0e30ad90f5
commit 60b423b741
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
modules/elasticsearch/src/main/java/org/elasticsearch/search/facet/filter/FilterFacetCollector.java
View File

@ -42,7 +42,7 @@ public class FilterFacetCollector extends AbstractFacetCollector {
public FilterFacetCollector(String facetName, Filter filter, FilterCache filterCache) { public FilterFacetCollector(String facetName, Filter filter, FilterCache filterCache) {
super(facetName); super(facetName);
this.filter = filterCache.cache(filter); this.filter = filter;
} }
@Override protected void doSetNextReader(IndexReader reader, int docBase) throws IOException { @Override protected void doSetNextReader(IndexReader reader, int docBase) throws IOException {

4
modules/elasticsearch/src/main/java/org/elasticsearch/search/facet/query/QueryFacetCollector.java
View File

@ -48,9 +48,9 @@ public class QueryFacetCollector extends AbstractFacetCollector {
super(facetName); super(facetName);
Filter possibleFilter = extractFilterIfApplicable(query); Filter possibleFilter = extractFilterIfApplicable(query);
if (possibleFilter != null) { if (possibleFilter != null) {
this.filter = filterCache.cache(possibleFilter); this.filter = possibleFilter;
} else { } else {
this.filter = filterCache.cache(new QueryWrapperFilter(query)); this.filter = new QueryWrapperFilter(query);
} }
} }