[
https://issues.apache.org/jira/browse/SOLR-10836?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jason Gerlowski updated SOLR-10836:
-----------------------------------
Attachment: SOLR-10836.patch
Saved you the trouble and ran tests on your change. All tests pass locally.
> 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
>
>
> 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]
