[ 
https://issues.apache.org/jira/browse/SOLR-8858?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Shalin Shekhar Mangar updated SOLR-8858:
----------------------------------------
    Attachment: SOLR-8858.patch

Condensed patch which applies to master. I added the following comment for the 
case where document cache is enabled:

{quote}
        // we do not pass the fields in this case because that would return an 
incomplete document which would
        // be eventually cached. The alternative would be to read the stored 
fields twice; once with the fields
        // and then without for caching leading to a performance hit
        // see SOLR-8858 for related discussion
{quote}

All tests and precommit passes. I'll commit this now.

> SolrIndexSearcher#doc() Completely Ignores Field Filters Unless Lazy Field 
> Loading is Enabled
> ---------------------------------------------------------------------------------------------
>
>                 Key: SOLR-8858
>                 URL: https://issues.apache.org/jira/browse/SOLR-8858
>             Project: Solr
>          Issue Type: Bug
>    Affects Versions: 4.6, 4.10, 5.5
>            Reporter: Caleb Rackliffe
>            Assignee: Shalin Shekhar Mangar
>              Labels: easyfix
>         Attachments: SOLR-8858.patch
>
>
> If {{enableLazyFieldLoading=false}}, a perfectly valid fields filter will be 
> ignored, and we'll create a {{DocumentStoredFieldVisitor}} without it.



--
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