Munendra S N commented on SOLR-11956:

Updated patch handles only Checked exception. Also, Throwing SolrException 
instead of IOException where it suits better.

One more, there are many assertions used in Collapse parser but assertions are 
not enabled by default and assertions would be ignored. I think either 
assertions should not be used or use of it should be documented

> Collapsing on undefined field returns 500
> -----------------------------------------
>                 Key: SOLR-11956
>                 URL: https://issues.apache.org/jira/browse/SOLR-11956
>             Project: Solr
>          Issue Type: Bug
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Munendra S N
>            Priority: Trivial
>         Attachments: SOLR-11956.patch, SOLR-11956.patch
> When collapsing is specified on the undefined field then, the response 
> returned has status 500 instead of 400. 
> This is because of wrapping of SolrException to RuntimeException
> https://github.com/apache/lucene-solr/blob/master/solr/core/src/java/org/apache/solr/search/CollapsingQParserPlugin.java#L377
> Then in request handler base, all RuntimeException are converted to 
> SolrException with status 500

This message was sent by Atlassian JIRA

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

Reply via email to