lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Erick Erickson (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (SOLR-9701) NPE in export handler when "fl" parameter is omitted.
Date Sun, 30 Oct 2016 02:48:59 GMT

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

Erick Erickson updated SOLR-9701:
---------------------------------
    Attachment: SOLR-9701.patch

Final patch with tests.

> NPE in export handler when "fl" parameter is omitted.
> -----------------------------------------------------
>
>                 Key: SOLR-9701
>                 URL: https://issues.apache.org/jira/browse/SOLR-9701
>             Project: Solr
>          Issue Type: Bug
>      Security Level: Public(Default Security Level. Issues are Public) 
>    Affects Versions: trunk, 6.4
>            Reporter: Erick Erickson
>            Assignee: Erick Erickson
>            Priority: Minor
>         Attachments: SOLR-9701.patch, SOLR-9701.patch
>
>
> This started when a user reported that if you do not specify any parameters for the export
handler, you get an NPE. I tracked it down to not specifying an "fl" parameter.
> But in general I rearranged the error reporting in SortingResponseWriter.write so that
immediately upon detecting a problem, the exception gets written to the output stream and
then return immediately rather than save it up for the end. Preliminary version of the patch
attached; it fixes the immediate problem.
> Still to see is if it breaks any tests since the first error detected will be returned
to the user rather than the last. I'll fix any tests that are sensitive to this and check
in sometime this weekend.



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

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


Mime
View raw message