You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by yo...@apache.org on 2012/02/26 17:34:20 UTC

svn commit: r1293866 - /lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/RecoveryStrategy.java

Author: yonik
Date: Sun Feb 26 16:34:19 2012
New Revision: 1293866

URL: http://svn.apache.org/viewvc?rev=1293866&view=rev
Log:
logging: fix detection of updates since startup

Modified:
    lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/RecoveryStrategy.java

Modified: lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/RecoveryStrategy.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/RecoveryStrategy.java?rev=1293866&r1=1293865&r2=1293866&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/RecoveryStrategy.java (original)
+++ lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/RecoveryStrategy.java Sun Feb 26 16:34:19 2012
@@ -229,10 +229,11 @@ public class RecoveryStrategy extends Th
         if (startingRecentVersions.get(oldIdx) == firstStartingVersion) break;
       }
 
-      if (oldIdx < startingRecentVersions.size()) {
-        log.info("####### Found new versions added after startup: num=" + (startingRecentVersions.size()-oldIdx));
+      if (oldIdx > 0) {
+        log.info("####### Found new versions added after startup: num=" + oldIdx);
       }
-      
+
+      // TODO: only log at debug level in the future (or move to oldIdx > 0 block)
       log.info("###### startupVersions=" + reallyStartingVersions);
       log.info("###### currentVersions=" + startingRecentVersions);
     }