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 2017/02/02 18:54:11 UTC
lucene-solr:branch_6x: SOLR-10018: default hl.maxAnalyazedChars to
51200 across all highlighters
Repository: lucene-solr
Updated Branches:
refs/heads/branch_6x e457613e6 -> 1c7ae87f0
SOLR-10018: default hl.maxAnalyazedChars to 51200 across all highlighters
(cherry picked from commit f5c6c3b)
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/1c7ae87f
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/1c7ae87f
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/1c7ae87f
Branch: refs/heads/branch_6x
Commit: 1c7ae87f0cbc3440d022fecfbb1f980bf244f4ce
Parents: e457613
Author: David Smiley <ds...@apache.org>
Authored: Thu Feb 2 13:51:39 2017 -0500
Committer: David Smiley <ds...@apache.org>
Committed: Thu Feb 2 13:54:05 2017 -0500
----------------------------------------------------------------------
solr/CHANGES.txt | 3 +++
.../org/apache/solr/highlight/DefaultSolrHighlighter.java | 3 +--
.../org/apache/solr/highlight/PostingsSolrHighlighter.java | 4 ++--
.../src/java/org/apache/solr/highlight/SolrHighlighter.java | 1 +
.../org/apache/solr/highlight/UnifiedSolrHighlighter.java | 8 ++++----
.../java/org/apache/solr/common/params/HighlightParams.java | 2 +-
6 files changed, 12 insertions(+), 9 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/1c7ae87f/solr/CHANGES.txt
----------------------------------------------------------------------
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index 28620df..91e4c9b 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -90,6 +90,9 @@ Other Changes
* SOLR-10043: Reduce logging of pre-start log rotation (janhoy)
+* SOLR-10018: Increase the default hl.maxAnalyzedChars to 51200 for the Unified & Postings Highlighter so that all
+ highlighters now have this same default. (David Smiley)
+
================== 6.4.1 ==================
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/1c7ae87f/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 e035a75..33ea575 100644
--- a/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java
+++ b/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java
@@ -525,8 +525,7 @@ public class DefaultSolrHighlighter extends SolrHighlighter implements PluginInf
}
int maxCharsToAnalyze = params.getFieldInt(fieldName,
- HighlightParams.MAX_CHARS,
- Highlighter.DEFAULT_MAX_CHARS_TO_ANALYZE);
+ HighlightParams.MAX_CHARS, DEFAULT_MAX_CHARS);
if (maxCharsToAnalyze < 0) {//e.g. -1
maxCharsToAnalyze = Integer.MAX_VALUE;
}
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/1c7ae87f/solr/core/src/java/org/apache/solr/highlight/PostingsSolrHighlighter.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/highlight/PostingsSolrHighlighter.java b/solr/core/src/java/org/apache/solr/highlight/PostingsSolrHighlighter.java
index 513b38a..9fcf9f3 100644
--- a/solr/core/src/java/org/apache/solr/highlight/PostingsSolrHighlighter.java
+++ b/solr/core/src/java/org/apache/solr/highlight/PostingsSolrHighlighter.java
@@ -66,7 +66,7 @@ import org.apache.solr.util.plugin.PluginInfoInitialized;
* <str name="hl.bs.country"></str>
* <str name="hl.bs.variant"></str>
* <str name="hl.bs.type">SENTENCE</str>
- * <int name="hl.maxAnalyzedChars">10000</int>
+ * <int name="hl.maxAnalyzedChars">51200</int>
* <str name="hl.multiValuedSeparatorChar"> </str>
* <bool name="hl.highlightMultiTerm">false</bool>
* </lst>
@@ -204,7 +204,7 @@ public class PostingsSolrHighlighter extends SolrHighlighter implements PluginIn
protected final IndexSchema schema;
public SolrExtendedPostingsHighlighter(SolrQueryRequest req) {
- super(req.getParams().getInt(HighlightParams.MAX_CHARS, PostingsHighlighter.DEFAULT_MAX_LENGTH));
+ super(req.getParams().getInt(HighlightParams.MAX_CHARS, DEFAULT_MAX_CHARS));
this.params = req.getParams();
this.schema = req.getSchema();
}
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/1c7ae87f/solr/core/src/java/org/apache/solr/highlight/SolrHighlighter.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/highlight/SolrHighlighter.java b/solr/core/src/java/org/apache/solr/highlight/SolrHighlighter.java
index e9ebf0c..a8ee734 100644
--- a/solr/core/src/java/org/apache/solr/highlight/SolrHighlighter.java
+++ b/solr/core/src/java/org/apache/solr/highlight/SolrHighlighter.java
@@ -31,6 +31,7 @@ import java.util.List;
public abstract class SolrHighlighter
{
+ public static int DEFAULT_MAX_CHARS = 51200;
public static int DEFAULT_PHRASE_LIMIT = 5000;
/**
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/1c7ae87f/solr/core/src/java/org/apache/solr/highlight/UnifiedSolrHighlighter.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/highlight/UnifiedSolrHighlighter.java b/solr/core/src/java/org/apache/solr/highlight/UnifiedSolrHighlighter.java
index 2633522..c80e522 100644
--- a/solr/core/src/java/org/apache/solr/highlight/UnifiedSolrHighlighter.java
+++ b/solr/core/src/java/org/apache/solr/highlight/UnifiedSolrHighlighter.java
@@ -62,8 +62,8 @@ import org.apache.solr.util.plugin.PluginInfoInitialized;
* <str name="hl.tag.post">&lt;/em&gt;</str>
* <str name="hl.simple.pre">&lt;em&gt;</str>
* <str name="hl.simple.post">&lt;/em&gt;</str>
- * <str name="hl.tag.ellipsis">... </str>
- * <bool name="hl.defaultSummary">true</bool>
+ * <str name="hl.tag.ellipsis">(internal/unspecified)</str>
+ * <bool name="hl.defaultSummary">false</bool>
* <str name="hl.encoder">simple</str>
* <float name="hl.score.k1">1.2</float>
* <float name="hl.score.b">0.75</float>
@@ -72,7 +72,7 @@ import org.apache.solr.util.plugin.PluginInfoInitialized;
* <str name="hl.bs.country"></str>
* <str name="hl.bs.variant"></str>
* <str name="hl.bs.type">SENTENCE</str>
- * <int name="hl.maxAnalyzedChars">10000</int>
+ * <int name="hl.maxAnalyzedChars">51200</int>
* <bool name="hl.highlightMultiTerm">true</bool>
* <bool name="hl.usePhraseHighlighter">true</bool>
* <int name="hl.cacheFieldValCharsThreshold">524288</int>
@@ -234,7 +234,7 @@ public class UnifiedSolrHighlighter extends SolrHighlighter implements PluginInf
this.params = req.getParams();
this.schema = req.getSchema();
this.setMaxLength(
- params.getInt(HighlightParams.MAX_CHARS, UnifiedHighlighter.DEFAULT_MAX_LENGTH));
+ params.getInt(HighlightParams.MAX_CHARS, DEFAULT_MAX_CHARS));
this.setCacheFieldValCharsThreshold(
params.getInt(HighlightParams.CACHE_FIELD_VAL_CHARS_THRESHOLD, DEFAULT_CACHE_CHARS_THRESHOLD));
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/1c7ae87f/solr/solrj/src/java/org/apache/solr/common/params/HighlightParams.java
----------------------------------------------------------------------
diff --git a/solr/solrj/src/java/org/apache/solr/common/params/HighlightParams.java b/solr/solrj/src/java/org/apache/solr/common/params/HighlightParams.java
index 997fc7e..57f6734 100644
--- a/solr/solrj/src/java/org/apache/solr/common/params/HighlightParams.java
+++ b/solr/solrj/src/java/org/apache/solr/common/params/HighlightParams.java
@@ -66,7 +66,7 @@ public interface HighlightParams {
// formatting
public static final String FORMATTER = HIGHLIGHT+".formatter"; // OH
- public static final String ENCODER = HIGHLIGHT+".encoder"; // OH, (UH, PH limited)
+ public static final String ENCODER = HIGHLIGHT+".encoder"; // all
public static final String MERGE_CONTIGUOUS_FRAGMENTS = HIGHLIGHT + ".mergeContiguous"; // OH
public static final String SIMPLE = "simple"; // OH
public static final String SIMPLE_PRE = HIGHLIGHT+"."+SIMPLE+".pre"; // OH