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

Hoss Man resolved SOLR-965.
---------------------------

    Resolution: Fixed
      Assignee: Hoss Man

thanks mark ... i added a little warning logging on init if hte deprecated 
feature was used, and tweaked your error message so it directed people to use 
PingRequestHandler instead of the old admin/pingQuery if they ever encountered 
that error.

Committed revision 769574.


> Better exception logging when no ping query configured
> ------------------------------------------------------
>
>                 Key: SOLR-965
>                 URL: https://issues.apache.org/jira/browse/SOLR-965
>             Project: Solr
>          Issue Type: Improvement
>    Affects Versions: 1.3
>            Reporter: Hoss Man
>            Assignee: Hoss Man
>             Fix For: 1.4
>
>         Attachments: SOLR-965.patch
>
>
> user noted that with the example multicore configs, 
> http://localhost:8983/solr/core1/admin/ping results in a 500 error and logs 
> quite a bit of crap starting with a NullPointerException.
> The failing ping is the correct behavior, because the example multicore 
> configs don't define a pingQuery -- but the code path of the failure and the 
> exceptions logged are less then ideal.  ping.jsp (which is still used if the 
> PingRequestHandler doesn't get registered) and SolrConfig.getPingQueryRequest 
> should me modified to log an (on topic) error and fail fast when there is no 
> <pingQuery> declaration.
> A quick skim of PingRequestHandlers suggests we may also want to add some 
> better logging (specifically: warnings about configuring it with good 
> defaults) if it gets called without any params.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to