You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sh...@apache.org on 2015/02/28 14:28:18 UTC

svn commit: r1662935 - in /lucene/dev/trunk/solr: CHANGES.txt core/src/java/org/apache/solr/cloud/OverseerAutoReplicaFailoverThread.java

Author: shalin
Date: Sat Feb 28 13:28:18 2015
New Revision: 1662935

URL: http://svn.apache.org/r1662935
Log:
SOLR-7178: OverseerAutoReplicaFailoverThread compares Integer objects using ==

Modified:
    lucene/dev/trunk/solr/CHANGES.txt
    lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerAutoReplicaFailoverThread.java

Modified: lucene/dev/trunk/solr/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/CHANGES.txt?rev=1662935&r1=1662934&r2=1662935&view=diff
==============================================================================
--- lucene/dev/trunk/solr/CHANGES.txt (original)
+++ lucene/dev/trunk/solr/CHANGES.txt Sat Feb 28 13:28:18 2015
@@ -167,7 +167,10 @@ Bug Fixes
   (Pablo Queixalos, shalin)
 
 * SOLR-7139: Fix SolrContentHandler for TIKA to ignore multiple startDocument events.
-  (Chris A. Mattmann, Uwe Schindler)  
+  (Chris A. Mattmann, Uwe Schindler)
+
+* SOLR-7178: OverseerAutoReplicaFailoverThread compares Integer objects using ==
+  (shalin)
 
 Optimizations
 ----------------------

Modified: lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerAutoReplicaFailoverThread.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerAutoReplicaFailoverThread.java?rev=1662935&r1=1662934&r2=1662935&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerAutoReplicaFailoverThread.java (original)
+++ lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerAutoReplicaFailoverThread.java Sat Feb 28 13:28:18 2015
@@ -152,7 +152,8 @@ public class OverseerAutoReplicaFailover
       return;
     }
     if (clusterState != null) {
-      if (lastClusterStateVersion == clusterState.getZkClusterStateVersion() && baseUrlForBadNodes.size() == 0 &&
+      if (clusterState.getZkClusterStateVersion() != null &&
+          clusterState.getZkClusterStateVersion().equals(lastClusterStateVersion) && baseUrlForBadNodes.size() == 0 &&
           liveNodes.equals(clusterState.getLiveNodes())) {
         // nothing has changed, no work to do
         return;