[ 
https://issues.apache.org/jira/browse/SOLR-2798?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Hoss Man updated SOLR-2798:
---------------------------
    Attachment: SOLR-2798.patch


Here's Demian's PR as a unified patch with a few more tweaks based on my 
review...

* deleted a really old/obsolte "note to self" Demian had acidently migrated 
somewhere it made even less sense
* added a few more asserts to QueryEqualityTest & DisMaxRequestHandlerTest for 
good measure
* noticed that QueryParsing.getLocalParams was still delegating to the old Map 
version
** added asserts to SimpleFacetsTest (using multiple facet.range.other 
localparams) to demonstrate how that still causes the bug in some cases
** fixed QueryParsing.getLocalParams so SimpleFacetsTest started passing again


Tests all pass. 

Still running precommit & then i'll move forwrad with trunk & start backporting

> Local Param parsing does not support multivalued params
> -------------------------------------------------------
>
>                 Key: SOLR-2798
>                 URL: https://issues.apache.org/jira/browse/SOLR-2798
>             Project: Solr
>          Issue Type: Bug
>            Reporter: Hoss Man
>            Assignee: Anshum Gupta
>         Attachments: SOLR-2798.patch
>
>
> As noted by Demian on the solr-user mailing list, Local Param parsing seems 
> to use a "last one wins" approach when parsing multivalued params.
> In this example, the value of "111" is completely ignored:
> {code}
> http://localhost:8983/solr/select?debug=query&q={!dismax%20bq=111%20bq=222}foo
> {code}



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to