From d289da2bad5ef265718f8c1674098e90cf4d92d3 Mon Sep 17 00:00:00 2001
From: Eric Pugh <epugh@opensourceconnections.com>
Date: Mon, 4 Mar 2024 08:59:10 -0500
Subject: [PATCH] kebab the replicationFactor

---
 solr/core/src/java/org/apache/solr/cli/CreateTool.java        | 4 ++--
 solr/core/src/java/org/apache/solr/cli/RunExampleTool.java    | 4 ++--
 .../src/test/org/apache/solr/cloud/SolrCloudExampleTest.java  | 2 +-
 .../deployment-guide/pages/solr-control-script-reference.adoc | 2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/solr/core/src/java/org/apache/solr/cli/CreateTool.java b/solr/core/src/java/org/apache/solr/cli/CreateTool.java
index af4bc0dda85..84acdb6469c 100644
--- a/solr/core/src/java/org/apache/solr/cli/CreateTool.java
+++ b/solr/core/src/java/org/apache/solr/cli/CreateTool.java
@@ -88,7 +88,7 @@ public List<Option> getOptions() {
             .desc("Number of shards; default is 1.")
             .build(),
         Option.builder("rf")
-            .longOpt("replicationFactor")
+            .longOpt("replication-factor")
             .argName("#")
             .hasArg()
             .required(false)
@@ -242,7 +242,7 @@ protected void createCollection(CloudSolrClient cloudSolrClient, CommandLine cli
     // build a URL to create the collection
     int numShards = Integer.parseInt(cli.getOptionValue("shards", String.valueOf(1)));
     int replicationFactor =
-        Integer.parseInt(cli.getOptionValue("replicationFactor", String.valueOf(1)));
+        Integer.parseInt(cli.getOptionValue("replication-factor", String.valueOf(1)));
 
     boolean configExistsInZk =
         confName != null
diff --git a/solr/core/src/java/org/apache/solr/cli/RunExampleTool.java b/solr/core/src/java/org/apache/solr/cli/RunExampleTool.java
index 8633e049af8..05ebc45388a 100644
--- a/solr/core/src/java/org/apache/solr/cli/RunExampleTool.java
+++ b/solr/core/src/java/org/apache/solr/cli/RunExampleTool.java
@@ -298,7 +298,7 @@ protected void runExample(CommandLine cli, String exampleName) throws Exception
           new String[] {
             "-name", collectionName,
             "-shards", "1",
-            "-replicationFactor", "1",
+            "--replication-factor", "1",
             "-confname", collectionName,
             "-confdir", configSet,
             "-configsetsDir", configsetsDir.getAbsolutePath(),
@@ -853,7 +853,7 @@ protected String createCloudExampleCollection(
         new String[] {
           "-name", collectionName,
           "-shards", String.valueOf(numShards),
-          "-replicationFactor", String.valueOf(replicationFactor),
+          "--replication-factor", String.valueOf(replicationFactor),
           "-confname", collectionName,
           "-confdir", cloudConfig,
           "-configsetsDir", configsetsDir.getAbsolutePath(),
diff --git a/solr/core/src/test/org/apache/solr/cloud/SolrCloudExampleTest.java b/solr/core/src/test/org/apache/solr/cloud/SolrCloudExampleTest.java
index 35f7949c01a..564f8383a1b 100644
--- a/solr/core/src/test/org/apache/solr/cloud/SolrCloudExampleTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/SolrCloudExampleTest.java
@@ -77,7 +77,7 @@ public void testLoadDocsIntoGettingStartedCollection() throws Exception {
           testCollectionName,
           "-shards",
           "2",
-          "-replicationFactor",
+          "--replication-factor",
           "2",
           "-confname",
           testCollectionName,
diff --git a/solr/solr-ref-guide/modules/deployment-guide/pages/solr-control-script-reference.adoc b/solr/solr-ref-guide/modules/deployment-guide/pages/solr-control-script-reference.adoc
index 67ec7d890f4..817cb19216d 100644
--- a/solr/solr-ref-guide/modules/deployment-guide/pages/solr-control-script-reference.adoc
+++ b/solr/solr-ref-guide/modules/deployment-guide/pages/solr-control-script-reference.adoc
@@ -708,7 +708,7 @@ Only applies when Solr is running in SolrCloud mode.
 +
 *Example*: `bin/solr create -s 2`
 
-`-rf <replicas>` or `--replicationFactor <replicas>`::
+`-rf <replicas>` or `--replication-factor <replicas>`::
 +
 [%autowidth,frame=none]
 |===