[
https://issues.apache.org/jira/browse/SOLR-5934?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Gregory Chanan updated SOLR-5934:
---------------------------------
Attachment: SOLR-5934.patch
> LBHttpSolrServer exception handling improvement and small test improvements
> ---------------------------------------------------------------------------
>
> Key: SOLR-5934
> URL: https://issues.apache.org/jira/browse/SOLR-5934
> Project: Solr
> Issue Type: Improvement
> Components: SolrCloud
> Affects Versions: 4.8, 5.0
> Reporter: Gregory Chanan
> Priority: Minor
> Attachments: SOLR-5934.patch
>
>
> The error handling in LBHttpSolrServer can be simplified -- right now almost
> identical code is run whether the server is a zombie or not, which sometimes
> doesn't make complete sense. For example, the zombie code goes through some
> effort to throw an exception or save the exception based on the type of
> exception, but the end result is the same -- an exception is thrown. It's
> simpler if the same code is run each time.
> Also, made some minor changes to test cases:
> - made sure SolrServer.shutdown is called in finally, so it happens even if a
> request throws an exception
> - got rid of some unnecessary checks
> - normalized some functions/variables so the functions are public scope and
> the variables aren't
--
This message was sent by Atlassian JIRA
(v6.2#6252)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
