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

Shalin Shekhar Mangar commented on SOLR-7689:
---------------------------------------------

bq.  I'll also hold off updating the CHANGES.txt until we know what release it 
will be in.

Creating the list of changes for 5.2.2 in the CHANGES.txt is part of the 
release process. So, please add this issue to the CHANGES.txt under 5.3 so that 
we can make a decision to backport this to 5.2.2. At least, this way this fix 
will not be forgotten even if you're on vacation or not paying attention.

> ReRankQuery rewrite method can change the QueryResultKey causing cache misses.
> ------------------------------------------------------------------------------
>
>                 Key: SOLR-7689
>                 URL: https://issues.apache.org/jira/browse/SOLR-7689
>             Project: Solr
>          Issue Type: Bug
>          Components: search, SearchComponents - other
>            Reporter: Emad Nashed
>            Assignee: Joel Bernstein
>             Fix For: Trunk, 5.2.2
>
>         Attachments: SOLR-7689.patch, SOLR-7689.patch, SOLR-7689.patch, 
> SOLR-7689.patch, SOLR-7689.patch
>
>
> In SolrIndexSearcher class, the key used to lookup results in 
> queryResultCache uses the original query. 
> However later in createNormalizedWeight the query gets re-written, and then 
> saved in the queryResultCache after it's re-written. 
> This causes cache misses for the same query, and un-necessary inserts in the 
> queryResultCache.
> I can reproduce this using a re-ranking query that is using a main query as a 
> dismax query, the dismax Query could be re-written into a TermQuery, which 
> makes sense, but will cause cache misses.
> I tested a quick solution by just using q.clone()  when it comes to build 
> QueryResultKey, and it works fine, but not sure if that is the best way of 
> doing it.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to