[ 
https://issues.apache.org/jira/browse/SOLR-9708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15662926#comment-15662926
 ] 

ASF GitHub Bot commented on SOLR-9708:
--------------------------------------

Github user dsmiley commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/107#discussion_r87743142
  
    --- Diff: 
solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java ---
    @@ -373,6 +373,11 @@ protected BoundaryScanner getBoundaryScanner(String 
fieldName, SolrParams params
         if (!isHighlightingEnabled(params)) // also returns early if no unique 
key field
           return null;
     
    +    boolean rewrite = query != null && 
!(Boolean.valueOf(params.get(HighlightParams.USE_PHRASE_HIGHLIGHTER, "true")) &&
    --- End diff --
    
    no biggie but I think a simple if(...) condition would be simpler; no 
variable.


> Expose UnifiedHighlighter in Solr
> ---------------------------------
>
>                 Key: SOLR-9708
>                 URL: https://issues.apache.org/jira/browse/SOLR-9708
>             Project: Solr
>          Issue Type: New Feature
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: highlighter
>            Reporter: Timothy M. Rodriguez
>            Assignee: David Smiley
>             Fix For: 6.4
>
>
> This ticket is for creating a Solr plugin that can utilize the new 
> UnifiedHighlighter which was initially committed in 
> https://issues.apache.org/jira/browse/LUCENE-7438



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to