catch throwable for tests (assert fails) - not just exception

git-svn-id: https://svn.apache.org/repos/asf/lucene/dev/branches/branch_4x@1363341 13f79535-47bb-0310-9956-ffa450edef68

solr/core/src/java/org/apache/solr/update/DefaultSolrCoreState.java

@@ -94,15 +94,15 @@ public synchronized void newIndexWriter(SolrCore core) throws IOException {
       } catch (InterruptedException e) {}
     }
 
-    if (indexWriter != null) {
-      try {
-        indexWriter.close();
-      } catch (Exception e) {
-        SolrException.log(log, "Error closing old IndexWriter", e);
-      }
-    }
-
     try {
+      if (indexWriter != null) {
+        try {
+          indexWriter.close();
+        } catch (Throwable t) {
+          SolrException.log(log, "Error closing old IndexWriter", t);
+        }
+      }
+
       indexWriter = createMainIndexWriter(core, "DirectUpdateHandler2", false,
           true);
       // we need to null this so it picks up the new writer next get call