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 2012/02/28 17:35:06 UTC

svn commit: r1294723 - /lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java

Author: markrmiller
Date: Tue Feb 28 16:35:05 2012
New Revision: 1294723

URL: http://svn.apache.org/viewvc?rev=1294723&view=rev
Log:
only add a close hook for the httpshardhandlerfactory if *we* created it

Modified:
    lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java

Modified: lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java?rev=1294723&r1=1294722&r2=1294723&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java (original)
+++ lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java Tue Feb 28 16:35:05 2012
@@ -142,16 +142,17 @@ public class SearchHandler extends Reque
       shardHandlerFactory = core.getCoreDescriptor().getCoreContainer().getShardHandlerFactory();
     } else {
       shardHandlerFactory = core.createInitInstance(shfInfo, ShardHandlerFactory.class, null, null);
+      core.addCloseHook(new CloseHook() {
+        @Override
+        public void preClose(SolrCore core) {
+          shardHandlerFactory.close();
+        }
+        @Override
+        public void postClose(SolrCore core) {
+        }
+      });
     }
-    core.addCloseHook(new CloseHook() {
-      @Override
-      public void preClose(SolrCore core) {
-        shardHandlerFactory.close();
-      }
-      @Override
-      public void postClose(SolrCore core) {
-      }
-    });
+
   }
 
   public List<SearchComponent> getComponents() {