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

Jan Høydahl commented on SOLR-9237:
-----------------------------------

I did a small test of subclassing {{DefaultSolrHighlighter}} and overriding 
{{doHighlightingByFastVectorHighlighter}}, accessing 
{{fvhContainer.fieldQuery}} and creating a new {{FvhContainer}}.

The only problem I saw was that IntelliJ did not like my subclass accessing 
{{fvhContainer.fieldQuery}}. It says *Cannot access 
org.apache.lucene.search.vectorhighlight.FieldQuery*. However, compiling with 
ant works.

[~covolution] do you have a chance to verify that the proposed patch solves 
your concerns?

> DefaultSolrHighlighter.doHighlightingByFastVectorHighlighter can't be 
> overidden
> -------------------------------------------------------------------------------
>
>                 Key: SOLR-9237
>                 URL: https://issues.apache.org/jira/browse/SOLR-9237
>             Project: Solr
>          Issue Type: Bug
>          Components: highlighter
>    Affects Versions: 6.1
>            Reporter: Gethin James
>            Assignee: Jan Høydahl
>             Fix For: 6.1.1, 6.2, master (7.0)
>
>         Attachments: SOLR-9237.patch
>
>
> With *6.1.0* the *protected* method 
> DefaultSolrHighlighter.doHighlightingByFastVectorHighlighter passes in a 
> *private* class called FvhContainer which makes it very difficult to extend.
> I have code that extends DefaultSolrHighlighter which I can't fix due to this 
> issue.
> Could you consider making FvhContainer  "protected" and use a constructor?



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to