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

ASF subversion and git services commented on SOLR-10304:
--------------------------------------------------------

Commit 9c423a61a4ab5d88103cb26c190864454c5c1cb0 in lucene-solr's branch 
refs/heads/branch_6x from [~dsmiley]
[ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=9c423a6 ]

SOLR-10304: Refactor new SolrDocumentFetcher out of SolrIndexSearcher

(cherry picked from commit f1aef3d)


> Refactor Document/Stored-field handling out of SolrIndexSearcher
> ----------------------------------------------------------------
>
>                 Key: SOLR-10304
>                 URL: https://issues.apache.org/jira/browse/SOLR-10304
>             Project: Solr
>          Issue Type: Task
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: David Smiley
>            Assignee: David Smiley
>         Attachments: SOLR_10304_SolrDocumentFetcher.patch, 
> SOLR_10304_SolrDocumentFetcher.patch
>
>
> SolrIndexSearcher is nearly 3 thousand lines of code.  A sizable part of it 
> pertains to Document handling, including various stored-field concerns and 
> docValue substitutions (docValueAsStored related).  There are already 
> comments marking the start and end of this part of SolrIndexSearcher, plus 
> there some fields and their initialization that are only in support of those 
> methods.  I propose that all of this go to a new companion class 
> {{SolrDocumentFetcher}}. SolrIndexSearcher can add a getter for it, and where 
> applicable existing callers can call to this instead.  "Override"'s will need 
> to stay of course.
> ( Originally proposed in SOLR-10286 )



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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

Reply via email to