Apache
/
lucene
mirror of https://github.com/apache/lucene.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

SOLR-15057: avoid unnecessary object retention in FacetRangeProcessor (#2160)

This commit is contained in:
Christine Poerschke 2021-01-07 18:45:46 +00:00 committed by GitHub
parent 6ff4a9b395
commit 60f2417aca
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 26 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
solr/CHANGES.txt
View File

@ -327,6 +327,8 @@ Other Changes
* SOLR-14951: Upgrade Angular JS 1.7.9 to 1.8.0 (Kevin Risden)
* SOLR-15057: Avoid unnecessary object retention in FacetRangeProcessor. (Christine Poerschke)
================== 8.7.0 ==================
Consult the lucene/CHANGES.txt file for additional, low level, changes in this release.

21
solr/core/src/java/org/apache/solr/search/facet/FacetRangeProcessor.java
View File

@ -484,19 +484,26 @@ class FacetRangeProcessor extends FacetProcessor<FacetRange> {
@SuppressWarnings({"unchecked", "rawtypes"})
private SimpleOrderedMap getRangeCountsIndexed() throws IOException {
final boolean hasSubFacets = !freq.getSubFacets().isEmpty();
int slotCount = rangeList.size() + otherList.size();
if (hasSubFacets) {
intersections = new DocSet[slotCount];
filters = new Query[slotCount];
} else {
intersections = null;
filters = null;
}
createAccs(fcontext.base.size(), slotCount);
for (int idx = 0; idx<rangeList.size(); idx++) {
rangeStats(rangeList.get(idx), idx);
rangeStats(rangeList.get(idx), idx, hasSubFacets);
}
for (int idx = 0; idx<otherList.size(); idx++) {
rangeStats(otherList.get(idx), rangeList.size() + idx);
rangeStats(otherList.get(idx), rangeList.size() + idx, hasSubFacets);
}
@ -511,7 +518,7 @@ class FacetRangeProcessor extends FacetProcessor<FacetRange> {
buckets.add(bucket);
bucket.add("val", range.label);
addStats(bucket, idx);
doSubs(bucket, idx);
if (hasSubFacets) doSubs(bucket, idx);
}
for (int idx = 0; idx<otherList.size(); idx++) {
@ -520,7 +527,7 @@ class FacetRangeProcessor extends FacetProcessor<FacetRange> {
SimpleOrderedMap bucket = new SimpleOrderedMap();
res.add(range.label.toString(), bucket);
addStats(bucket, rangeList.size() + idx);
doSubs(bucket, rangeList.size() + idx);
if (hasSubFacets) doSubs(bucket, rangeList.size() + idx);
}
if (null != actual_end) {
@ -532,7 +539,7 @@ class FacetRangeProcessor extends FacetProcessor<FacetRange> {
private Query[] filters;
private DocSet[] intersections;
private void rangeStats(Range range, int slot) throws IOException {
private void rangeStats(Range range, int slot, boolean hasSubFacets) throws IOException {
final Query rangeQ;
{
final Query rangeQuery = sf.getType().getRangeQuery(null, sf, range.low == null ? null : calc.formatValue(range.low), range.high==null ? null : calc.formatValue(range.high), range.includeLower, range.includeUpper);
@ -549,8 +556,10 @@ class FacetRangeProcessor extends FacetProcessor<FacetRange> {
}
// TODO: specialize count only
DocSet intersection = fcontext.searcher.getDocSet(rangeQ, fcontext.base);
if (hasSubFacets) {
filters[slot] = rangeQ;
intersections[slot] = intersection; // save for later // TODO: only save if number of slots is small enough?
}
long num = collect(intersection, slot, slotNum -> { return new SlotAcc.SlotContext(rangeQ); });
countAcc.incrementCount(slot, num); // TODO: roll this into collect()
}
@ -558,7 +567,6 @@ class FacetRangeProcessor extends FacetProcessor<FacetRange> {
@SuppressWarnings({"unchecked", "rawtypes"})
private void doSubs(SimpleOrderedMap bucket, int slot) throws IOException {
// handle sub-facets for this bucket
if (freq.getSubFacets().size() > 0) {
DocSet subBase = intersections[slot];
try {
processSubs(bucket, filters[slot], subBase, false, null);
@ -567,7 +575,6 @@ class FacetRangeProcessor extends FacetProcessor<FacetRange> {
// subContext.base = null; // do not modify context after creation... there may be deferred execution (i.e. streaming)
}
}
}
// Essentially copied from SimpleFacets...
// would be nice to unify this stuff w/ analytics component...