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

David Smiley commented on SOLR-12562:
-------------------------------------

Can you see if we can remove {{materialize()}} altogether by always using the 
logic {{schemaField.getType().toObject(indexableField)}}?

If we retain the two methods specified in the issue title, I think we should 
add a comment in both places referring to each other to acknowledge the similar 
code.  It'll help us maintain this in the future.

> Remove redundant RealTimeGetCompnent.toSolrDoc and use 
> DocStreamer.convertLuceneDocToSolrDoc
> --------------------------------------------------------------------------------------------
>
>                 Key: SOLR-12562
>                 URL: https://issues.apache.org/jira/browse/SOLR-12562
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Erick Erickson
>            Assignee: Erick Erickson
>            Priority: Minor
>         Attachments: SOLR-12562.patch
>
>
> This code looks really redundant so we should remove one. The one in 
> RealTimeGet is the only used locally so my vote is to remove that one.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to