You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by ry...@apache.org on 2008/12/19 18:56:07 UTC

svn commit: r728082 - /lucene/solr/trunk/src/java/org/apache/solr/core/SolrCore.java

Author: ryan
Date: Fri Dec 19 09:56:07 2008
New Revision: 728082

URL: http://svn.apache.org/viewvc?rev=728082&view=rev
Log:
SOLR-930 -- reverting

Modified:
    lucene/solr/trunk/src/java/org/apache/solr/core/SolrCore.java

Modified: lucene/solr/trunk/src/java/org/apache/solr/core/SolrCore.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/core/SolrCore.java?rev=728082&r1=728081&r2=728082&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/core/SolrCore.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/core/SolrCore.java Fri Dec 19 09:56:07 2008
@@ -698,10 +698,7 @@
    */
   public void close() {
     int count = refCount.decrementAndGet();
-    if (count > 0){
-      log.warn( "Attempted close on {} did not succeed because the new reference count {} is > 0. ", this, count );
-      return;
-    }
+    if (count > 0) return; // close is called often, and only actually closes if nothing is using it.
     if (count < 0) {
       log.error("Too many close [count:{}] on {}. Please report this exception to solr-user@lucene.apache.org", count, this );
       return;