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();
}
}