You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ds...@apache.org on 2018/11/08 19:50:04 UTC
lucene-solr:branch_7x: SOLR-12964: Use
DocValuesIterator.advanceExact() instead of the advance()/docID() pattern
Repository: lucene-solr
Updated Branches:
refs/heads/branch_7x e5083e28f -> 9c4dc4d60
SOLR-12964: Use DocValuesIterator.advanceExact() instead of the advance()/docID() pattern
Closes #493
(cherry picked from commit 243a8a668a613835e3c114f6e5c31a098699840e)
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/9c4dc4d6
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/9c4dc4d6
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/9c4dc4d6
Branch: refs/heads/branch_7x
Commit: 9c4dc4d60a5d50592ae7718e9d407cb0c67f46cb
Parents: e5083e2
Author: Tim Underwood <ti...@gmail.com>
Authored: Thu Nov 8 14:34:17 2018 -0500
Committer: David Smiley <ds...@apache.org>
Committed: Thu Nov 8 14:35:46 2018 -0500
----------------------------------------------------------------------
solr/CHANGES.txt | 3 +++
.../apache/solr/search/facet/FacetFieldProcessorByHashDV.java | 5 +----
.../org/apache/solr/search/facet/UniqueSinglevaluedSlotAcc.java | 5 +----
3 files changed, 5 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/9c4dc4d6/solr/CHANGES.txt
----------------------------------------------------------------------
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index 0e3477f..a0e8929 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -205,6 +205,9 @@ Improvements
* LUCENE-8557: Some internal LeafReader.getFieldInfos implementations were being re-computed on-demand instead of
once up front leading to some slowdowns in places like JSON Facets and field collapsing. (Tim Underwood, David Smiley)
+* SOLR-12964: Json Facets: use DocValuesIterator advanceExact() instead of advance() in FacetFieldProcessorByHashDV and
+ UniqueSinglevaluedSlotAcc. (Tim Underwood)
+
================== 7.5.0 ==================
Consult the LUCENE_CHANGES.txt file for additional, low level, changes in this release.
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/9c4dc4d6/solr/core/src/java/org/apache/solr/search/facet/FacetFieldProcessorByHashDV.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/search/facet/FacetFieldProcessorByHashDV.java b/solr/core/src/java/org/apache/solr/search/facet/FacetFieldProcessorByHashDV.java
index 109e3d4..cffe0f7 100644
--- a/solr/core/src/java/org/apache/solr/search/facet/FacetFieldProcessorByHashDV.java
+++ b/solr/core/src/java/org/apache/solr/search/facet/FacetFieldProcessorByHashDV.java
@@ -360,10 +360,7 @@ class FacetFieldProcessorByHashDV extends FacetFieldProcessor {
@Override
public void collect(int segDoc) throws IOException {
- if (segDoc > docValues.docID()) {
- docValues.advance(segDoc);
- }
- if (segDoc == docValues.docID()) {
+ if (docValues.advanceExact(segDoc)) {
long val = toGlobal.get(docValues.ordValue());
collectValFirstPhase(segDoc, val);
}
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/9c4dc4d6/solr/core/src/java/org/apache/solr/search/facet/UniqueSinglevaluedSlotAcc.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/search/facet/UniqueSinglevaluedSlotAcc.java b/solr/core/src/java/org/apache/solr/search/facet/UniqueSinglevaluedSlotAcc.java
index ed51c5b..6057dd0 100644
--- a/solr/core/src/java/org/apache/solr/search/facet/UniqueSinglevaluedSlotAcc.java
+++ b/solr/core/src/java/org/apache/solr/search/facet/UniqueSinglevaluedSlotAcc.java
@@ -75,10 +75,7 @@ class UniqueSinglevaluedSlotAcc extends UniqueSlotAcc {
@Override
public void collect(int doc, int slotNum, IntFunction<SlotContext> slotContext) throws IOException {
- if (doc > subDv.docID()) {
- subDv.advance(doc);
- }
- if (doc == subDv.docID()) {
+ if (subDv.advanceExact(doc)) {
int segOrd = subDv.ordValue();
int ord = toGlobal==null ? segOrd : (int)toGlobal.get(segOrd);