You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ja...@apache.org on 2016/06/26 22:40:04 UTC
lucene-solr:branch_6x: SOLR-9237:
DefaultSolrHighlighter.doHighlightingByFastVectorHighlighter can't be
overidden (cherry picked from commit 7eb7702)
Repository: lucene-solr
Updated Branches:
refs/heads/branch_6x d1a047ad6 -> 319f89210
SOLR-9237: DefaultSolrHighlighter.doHighlightingByFastVectorHighlighter can't be overidden
(cherry picked from commit 7eb7702)
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/319f8921
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/319f8921
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/319f8921
Branch: refs/heads/branch_6x
Commit: 319f89210ac29c53828bd16e1a77e01cd08c164b
Parents: d1a047a
Author: Jan H�ydahl <ja...@apache.org>
Authored: Sun Jun 26 23:59:59 2016 +0200
Committer: Jan H�ydahl <ja...@apache.org>
Committed: Mon Jun 27 00:01:09 2016 +0200
----------------------------------------------------------------------
solr/CHANGES.txt | 2 ++
.../solr/highlight/DefaultSolrHighlighter.java | 15 ++++++++++-----
2 files changed, 12 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/319f8921/solr/CHANGES.txt
----------------------------------------------------------------------
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index 3e5e05d..88ca2e3 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -50,6 +50,8 @@ Bug Fixes
* SOLR-8546: SOLR-8546: TestLazyCores is failing a lot on the Jenkins cluster. (Erick Erickson)
+* SOLR-9237: DefaultSolrHighlighter.doHighlightingByFastVectorHighlighter can't be overidden (janhoy)
+
Optimizations
----------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/319f8921/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java b/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java
index 9941b16..ca481cd 100644
--- a/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java
+++ b/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java
@@ -389,7 +389,7 @@ public class DefaultSolrHighlighter extends SolrHighlighter implements PluginInf
preFetchFieldNames.add(keyField.getName());
}
- FvhContainer fvhContainer = new FvhContainer(); // Lazy container for fvh and fieldQuery
+ FvhContainer fvhContainer = new FvhContainer(null, null); // Lazy container for fvh and fieldQuery
IndexReader reader = new TermVectorReusingLeafReader(req.getSearcher().getLeafReader()); // SOLR-5855
@@ -422,7 +422,7 @@ public class DefaultSolrHighlighter extends SolrHighlighter implements PluginInf
return fragments;
}
- private Object doHighlightingOfField(Document doc, int docId, SchemaField schemaField,
+ protected Object doHighlightingOfField(Document doc, int docId, SchemaField schemaField,
FvhContainer fvhContainer, Query query, IndexReader reader, SolrQueryRequest req,
SolrParams params) throws IOException {
Object fieldHighlights;
@@ -740,9 +740,14 @@ public class DefaultSolrHighlighter extends SolrHighlighter implements PluginInf
}
// Wraps FVH to allow pass-by-reference
- private class FvhContainer {
- private FastVectorHighlighter fvh;
- private FieldQuery fieldQuery;
+ protected class FvhContainer {
+ FastVectorHighlighter fvh;
+ FieldQuery fieldQuery;
+
+ FvhContainer(FastVectorHighlighter fvh, FieldQuery fieldQuery) {
+ this.fvh = fvh;
+ this.fieldQuery = fieldQuery;
+ }
}
}