You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Cassandra Targett (JIRA)" <ji...@apache.org> on 2017/11/10 22:12:00 UTC
[jira] [Updated] (SOLR-4107) Parse Error message lost from 3.5.0 to
3.6.1
[ https://issues.apache.org/jira/browse/SOLR-4107?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Cassandra Targett updated SOLR-4107:
------------------------------------
Component/s: (was: search)
clients - java
> Parse Error message lost from 3.5.0 to 3.6.1
> --------------------------------------------
>
> Key: SOLR-4107
> URL: https://issues.apache.org/jira/browse/SOLR-4107
> Project: Solr
> Issue Type: Bug
> Components: clients - java
> Affects Versions: 3.6.1
> Reporter: Pierre Gossé
> Priority: Minor
>
> QueryComponent.prepare builds a SolrException from ParseException encountered, using a constructor that forces error message to null
> in 3.5.0
> public SolrException(ErrorCode code, Throwable th) {
> super(th);
> this.code=code.code;
> logged=true;
> }
> in 3.6.1 :
> public SolrException(ErrorCode code, Throwable th) {
> this(code, null, th, (th instanceof SolrException) ? ((SolrException)th).logged : false);
> }
> calling :
> public SolrException(ErrorCode code, String msg, Throwable th, boolean alreadyLogged) {
> super(msg,th);
> this.code=code.code;
> logged=alreadyLogged;
> }
> I don't think this is a desired behaviour, so I guessed this should be corrected by changing line 93 to
> this(code, th.getMessage(), th, (th instanceof SolrException) ?(SolrException)th).logged : false);
> I'll try to add a patch later today.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org