You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by rm...@apache.org on 2010/10/17 23:13:40 UTC

svn commit: r1023583 - /lucene/dev/trunk/solr/src/test/org/apache/solr/core/TestQuerySenderListener.java

Author: rmuir
Date: Sun Oct 17 21:13:39 2010
New Revision: 1023583

URL: http://svn.apache.org/viewvc?rev=1023583&view=rev
Log:
clean up searchers

Modified:
    lucene/dev/trunk/solr/src/test/org/apache/solr/core/TestQuerySenderListener.java

Modified: lucene/dev/trunk/solr/src/test/org/apache/solr/core/TestQuerySenderListener.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/src/test/org/apache/solr/core/TestQuerySenderListener.java?rev=1023583&r1=1023582&r2=1023583&view=diff
==============================================================================
--- lucene/dev/trunk/solr/src/test/org/apache/solr/core/TestQuerySenderListener.java (original)
+++ lucene/dev/trunk/solr/src/test/org/apache/solr/core/TestQuerySenderListener.java Sun Oct 17 21:13:39 2010
@@ -19,6 +19,7 @@ package org.apache.solr.core;
 
 import org.apache.solr.SolrTestCaseJ4;
 import org.apache.solr.search.SolrIndexSearcher;
+import org.apache.solr.util.RefCounted;
 import org.apache.solr.common.params.EventParams;
 import org.apache.lucene.store.Directory;
 import org.junit.BeforeClass;
@@ -48,7 +49,8 @@ public class TestQuerySenderListener ext
     assertTrue("Not an instance of QuerySenderListener", newSearcherListener instanceof QuerySenderListener);
     QuerySenderListener qsl = (QuerySenderListener) newSearcherListener;
 
-    SolrIndexSearcher currentSearcher = core.getSearcher().get();
+    RefCounted<SolrIndexSearcher> currentSearcherRef = core.getSearcher();
+    SolrIndexSearcher currentSearcher = currentSearcherRef.get();
     qsl.newSearcher(currentSearcher, null);//test new Searcher
     MockQuerySenderListenerReqHandler mock = (MockQuerySenderListenerReqHandler) core.getRequestHandler("mock");
     assertNotNull("Mock is null", mock);
@@ -62,6 +64,8 @@ public class TestQuerySenderListener ext
     evt = mock.req.getParams().get(EventParams.EVENT);
     assertNotNull("Event is null", evt);
     assertTrue(evt + " is not equal to " + EventParams.NEW_SEARCHER, evt.equals(EventParams.NEW_SEARCHER) == true);
+    newSearcher.close();
+    currentSearcherRef.decref();
   }
 
 }