From 6215e6414a584f73c53ca0cf591d0e55c40a5297 Mon Sep 17 00:00:00 2001
From: Serhiy Bzhezytskyy <me@serhiy-bzhezytskyy.com>
Date: Wed, 19 Aug 2026 20:50:58 +0300
Subject: [PATCH] SOLR-18383: remove CollectionParams.FROM_NODE

Deprecated since 10.0 in favor of SOURCE_NODE. 0 remaining references
after removing the constant and its 2 usages in MoveReplicaCmd/CollectionsHandler.

AI-assisted (Claude Sonnet 5)
---
 .../SOLR-18383-remove-collectionparams-from-node.yml     | 8 ++++++++
 .../solr/cloud/api/collections/MoveReplicaCmd.java       | 9 ++-------
 .../apache/solr/handler/admin/CollectionsHandler.java    | 1 -
 .../org/apache/solr/common/params/CollectionParams.java  | 5 -----
 4 files changed, 10 insertions(+), 13 deletions(-)
 create mode 100644 changelog/unreleased/SOLR-18383-remove-collectionparams-from-node.yml

diff --git a/changelog/unreleased/SOLR-18383-remove-collectionparams-from-node.yml b/changelog/unreleased/SOLR-18383-remove-collectionparams-from-node.yml
new file mode 100644
index 000000000000..42398be2a1f2
--- /dev/null
+++ b/changelog/unreleased/SOLR-18383-remove-collectionparams-from-node.yml
@@ -0,0 +1,8 @@
+# See https://github.com/apache/solr/blob/main/dev-docs/changelog.adoc
+title: Remove the deprecated `fromNode` request parameter of the MOVEREPLICA Collections API command; use `sourceNode` instead.
+type: removed
+authors:
+  - name: Serhiy Bzhezytskyy
+links:
+  - name: SOLR-18383
+    url: https://issues.apache.org/jira/browse/SOLR-18383
diff --git a/solr/core/src/java/org/apache/solr/cloud/api/collections/MoveReplicaCmd.java b/solr/core/src/java/org/apache/solr/cloud/api/collections/MoveReplicaCmd.java
index 634b23746335..635bae1e6821 100644
--- a/solr/core/src/java/org/apache/solr/cloud/api/collections/MoveReplicaCmd.java
+++ b/solr/core/src/java/org/apache/solr/cloud/api/collections/MoveReplicaCmd.java
@@ -108,16 +108,11 @@ private void moveReplica(
             "Collection: " + collection + " replica: " + replicaName + " does not exist");
       }
     } else {
-      String sourceNode =
-          message.getStr(CollectionParams.SOURCE_NODE, message.getStr(CollectionParams.FROM_NODE));
+      String sourceNode = message.getStr(CollectionParams.SOURCE_NODE);
       if (sourceNode == null) {
         throw new SolrException(
             SolrException.ErrorCode.BAD_REQUEST,
-            "'"
-                + CollectionParams.SOURCE_NODE
-                + " or '"
-                + CollectionParams.FROM_NODE
-                + "' is a required param");
+            "'" + CollectionParams.SOURCE_NODE + "' is a required param");
       }
       String shardId = message.getStr(SHARD_ID_PROP);
       if (shardId == null) {
diff --git a/solr/core/src/java/org/apache/solr/handler/admin/CollectionsHandler.java b/solr/core/src/java/org/apache/solr/handler/admin/CollectionsHandler.java
index 6e3c9e453d78..270afc24906f 100644
--- a/solr/core/src/java/org/apache/solr/handler/admin/CollectionsHandler.java
+++ b/solr/core/src/java/org/apache/solr/handler/admin/CollectionsHandler.java
@@ -1180,7 +1180,6 @@ public Map<String, Object> execute(
           return copy(
               req.getParams(),
               map,
-              CollectionParams.FROM_NODE,
               CollectionParams.SOURCE_NODE,
               TARGET_NODE,
               WAIT_FOR_FINAL_STATE,
diff --git a/solr/solrj/src/java/org/apache/solr/common/params/CollectionParams.java b/solr/solrj/src/java/org/apache/solr/common/params/CollectionParams.java
index 3e1d9745c93e..14b85ef94a44 100644
--- a/solr/solrj/src/java/org/apache/solr/common/params/CollectionParams.java
+++ b/solr/solrj/src/java/org/apache/solr/common/params/CollectionParams.java
@@ -30,11 +30,6 @@ public interface CollectionParams {
 
   String NAME = "name";
 
-  /**
-   * @deprecated use {@link #SOURCE_NODE} instead
-   */
-  @Deprecated String FROM_NODE = "fromNode";
-
   String SOURCE_NODE = "sourceNode";
   String TARGET_NODE = "targetNode";
   String SOURCE_NODES = "sourceNodes";