[
https://issues.apache.org/jira/browse/SOLR-14364?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17074967#comment-17074967
]
ASF subversion and git services commented on SOLR-14364:
--------------------------------------------------------
Commit 7b3980c0804b0de16841faf16478b2b1bd217088 in lucene-solr's branch
refs/heads/master from David Smiley
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=7b3980c ]
SOLR-14364: LTR SolrFeature fq improvements
Mostly general code improvements, though it should support postFilters now
Add QueryUtils.combineQueryAndFilter
> LTR SolrFeature fq improvements, like PostFilters
> -------------------------------------------------
>
> Key: SOLR-14364
> URL: https://issues.apache.org/jira/browse/SOLR-14364
> Project: Solr
> Issue Type: Improvement
> Security Level: Public(Default Security Level. Issues are Public)
> Components: contrib - LTR
> Reporter: David Smiley
> Assignee: David Smiley
> Priority: Major
> Time Spent: 3h 10m
> Remaining Estimate: 0h
>
> While working on another issue, I came upon LTR's SolrFeature class which
> calls SolrIndexSearcher.getProcessedFilter – a rather low-level method that
> that nobody should call directly if they can help it. Furthermore, the way
> SolrFeature was processing the filter queries didn't support PostFilters,
> which would have been silently ignored (ouch!). After looking around this
> SolrFeature class I found multiple things to improve to reduce complexity,
> perhaps performance in some cases, and IMO improve readability. In my PR it
> has about 80 fewer lines of code now. Additionally I found it useful to add
> a new utility method combineQueryAndFilter that is useful in quite a few
> places across Solr (and my PR does so).
> I have some complaints with other things I see but want to stop here, with my
> primary objective of reducing a needless caller on getProcessedFilter.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
