[
https://issues.apache.org/jira/browse/SOLR-10836?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
David Smiley updated SOLR-10836:
--------------------------------
Attachment:
SOLR_10836_use_SolrIndexSearcher_cached_MultiFields_instead_of_recreating.patch
The attached patch is for several parsers that had the same code. I also fixed
the null Terms bug... I should probably reclassify this issue as a bug fix.
> TextLogisticRegressionQParserPlugin needn't call MultiFields; use
> SolrIndexSearcher's cached instance
> -----------------------------------------------------------------------------------------------------
>
> Key: SOLR-10836
> URL: https://issues.apache.org/jira/browse/SOLR-10836
> Project: Solr
> Issue Type: Improvement
> Security Level: Public(Default Security Level. Issues are Public)
> Reporter: David Smiley
> Priority: Minor
> Attachments: SOLR-10836.patch,
> SOLR_10836_use_SolrIndexSearcher_cached_MultiFields_instead_of_recreating.patch
>
>
> org.apache.solr.search.TextLogisticRegressionQParserPlugin.TextLogisticRegressionCollector#finish
> contains this line of code:
> {code:java}
> Terms terms =
> MultiFields.getFields(searcher.getIndexReader()).terms(trainingParams.feature);
> {code}
> Instead I think it should be this:
> {code:java}
> Terms terms =
> ((SolrIndexSearcher)searcher).getSlowAtomicReader().terms(trainingParams.feature);
> {code}
> MultiFields can take some time to build and Solr (via SlowAtomicReader) has
> already done this work.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
