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

Erick Erickson commented on SOLR-5057:
--------------------------------------

Hmmm, sorry it took so long. But it appears that your latest patch doesn't have 
any of Yonik's suggestions in it and still has the test in a completely new 
file rather than just another test case in QueryResultKeyTest.java as my patch 
had. Did you upload your first patch again by mistake? 

Please upload a new version and I _promise_ I'll get it committed soon.
                
> queryResultCache should not related with the order of fq's list
> ---------------------------------------------------------------
>
>                 Key: SOLR-5057
>                 URL: https://issues.apache.org/jira/browse/SOLR-5057
>             Project: Solr
>          Issue Type: Improvement
>          Components: search
>    Affects Versions: 4.0, 4.1, 4.2, 4.3
>            Reporter: Feihong Huang
>            Assignee: Erick Erickson
>            Priority: Minor
>         Attachments: SOLR-5057.patch, SOLR-5057.patch, SOLR-5057.patch
>
>   Original Estimate: 48h
>  Remaining Estimate: 48h
>
> There are two case query with the same meaning below. But the case2 can't use 
> the queryResultCache when case1 is executed.
> case1: q=*:*&fq=field1:value1&fq=field2:value2
> case2: q=*:*&fq=field2:value2&fq=field1:value1
> I think queryResultCache should not be related with the order of fq's list.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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

Reply via email to