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

Jason Gerlowski commented on LUCENE-9026:
-----------------------------------------

I've committed this on master and branch_8x with some added Javadocs as Adrien 
suggested.  (Though admittedly the javadoc text doesn't give much information 
that the method names don't already share.)

Closing this out accordingly.  Thanks everyone for the reviews and feedback.

> Make it easier to extend DocValuesTermsQuery
> --------------------------------------------
>
>                 Key: LUCENE-9026
>                 URL: https://issues.apache.org/jira/browse/LUCENE-9026
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: core/search
>    Affects Versions: master (9.0)
>            Reporter: Jason Gerlowski
>            Assignee: Jason Gerlowski
>            Priority: Minor
>         Attachments: LUCENESOLR-9026.patch
>
>
> The visibility of some of the fields in DocValuesTermsQuery make it difficult 
> to efficiently subclass.  Especially the "termData" instance variable, which 
> is really core to the functioning of the class but is totally inaccessible 
> from any sub-classes, forcing subclasses to store a duplicate 
> PrefixCodedTerms object, and then juggle the state of both.
> Are there any objections to making "termData" (and potentiall some other 
> instance variables) protected instead of private for this class?



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to