opendev/gerrit
Code Issues Proposed changes

Only load legacy field when reading the change document

Browse Source 
The only stored field used is the legacy Change.Id field, so
tell Lucene this is the only field to unpack and return.

Change-Id: Ic387ce33433b288062e96836afc672313175e7da
This commit is contained in:
Shawn Pearce
2013-06-25 23:00:44 -06:00
parent 371db40f78
commit da89859da4
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneChangeIndex.java
View File

@@ -20,6 +20,7 @@ import static org.apache.lucene.search.BooleanClause.Occur.MUST;
import static org.apache.lucene.search.BooleanClause.Occur.MUST_NOT;
import static org.apache.lucene.search.BooleanClause.Occur.SHOULD;
import com.google.common.collect.ImmutableSet;
import com.google.common.collect.Lists;
import com.google.common.collect.Sets;
import com.google.common.util.concurrent.Futures;
@@ -347,9 +348,10 @@ public class LuceneChangeIndex implements ChangeIndex, LifecycleListener {
return new PrefixQuery(new Term(p.getField().getName(), p.getValue()));
}
private class QuerySource implements ChangeDataSource {
private static class QuerySource implements ChangeDataSource {
// TODO(dborowitz): Push limit down from predicate tree.
private static final int LIMIT = 1000;
private static final ImmutableSet<String> FIELDS = ImmutableSet.of(ID_FIELD);
private final List<SubIndex> indexes;
private final Query query;
@@ -388,7 +390,7 @@ public class LuceneChangeIndex implements ChangeIndex, LifecycleListener {
List<ChangeData> result =
Lists.newArrayListWithCapacity(docs.scoreDocs.length);
for (ScoreDoc sd : docs.scoreDocs) {
Document doc = searchers[sd.shardIndex].doc(sd.doc);
Document doc = searchers[sd.shardIndex].doc(sd.doc, FIELDS);
Number v = doc.getField(ID_FIELD).numericValue();
result.add(new ChangeData(new Change.Id(v.intValue())));
}