[SOLR-11449] MoveReplicaCmd mistakenly calls registerCollectionStateWatcher instead of unregisterCollectionStateWatcher

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 7.0.1, 7.1, 8.0
• Fix Version/s: 7.1, 8.0
• Assignee: Andrzej Bialecki
• Reporter: Andrzej Bialecki
• Created: 09/Oct/17 13:45
• Updated: 08/Jun/19 15:16
• Resolved: 10/Oct/17 10:50

Description

Probably got in as a copy & paste error:

--- a/solr/core/src/java/org/apache/solr/cloud/MoveReplicaCmd.java
+++ b/solr/core/src/java/org/apache/solr/cloud/MoveReplicaCmd.java
@@ -206,7 +206,7 @@ public class MoveReplicaCmd implements Cmd{
       log.warn(errorString);
       results.add("failure", errorString);
       if (watcher != null) { // unregister
-        ocmh.zkStateReader.registerCollectionStateWatcher(coll.getName(), watcher);
+        ocmh.zkStateReader.removeCollectionStateWatcher(coll.getName(), watcher);
       }
       return;
     }