You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ma...@apache.org on 2013/04/20 19:16:26 UTC
svn commit: r1470198 -
/lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java
Author: markrmiller
Date: Sat Apr 20 17:16:25 2013
New Revision: 1470198
URL: http://svn.apache.org/r1470198
Log:
don't return in a finally block
Modified:
lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java
Modified: lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java?rev=1470198&r1=1470197&r2=1470198&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java (original)
+++ lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java Sat Apr 20 17:16:25 2013
@@ -203,17 +203,17 @@ public class OverseerCollectionProcessor
+ operation);
}
- } catch (Exception ex) {
+ } catch (Throwable t) {
SolrException.log(log, "Collection " + operation + " of " + operation
- + " failed", ex);
- results.add("Operation " + operation + " caused exception:", ex);
+ + " failed", t);
+ results.add("Operation " + operation + " caused exception:", t);
SimpleOrderedMap nl = new SimpleOrderedMap();
- nl.add("msg", ex.getMessage());
- nl.add("rspCode", ex instanceof SolrException ? ((SolrException)ex).code() : -1);
+ nl.add("msg", t.getMessage());
+ nl.add("rspCode", t instanceof SolrException ? ((SolrException)t).code() : -1);
results.add("exception", nl);
- } finally {
- return new OverseerSolrResponse(results);
- }
+ }
+
+ return new OverseerSolrResponse(results);
}
private void createAlias(Aliases aliases, ZkNodeProps message) {