malliaridis commented on code in PR #2710:
URL: https://github.com/apache/solr/pull/2710#discussion_r1779652256


##########
solr/core/src/java/org/apache/solr/cli/DeleteTool.java:
##########
@@ -122,16 +124,36 @@ public List<Option> getOptions() {
             .desc(
                 "Skip safety checks when deleting the configuration directory 
used by a collection.")
             .build(),
+        Option.builder("p")
+            .longOpt("port")
+            .deprecated(
+                DeprecatedAttributes.builder()
+                    .setForRemoval(true)
+                    .setSince("9.7")
+                    .setDescription("Use --solr-url instead")
+                    .get())
+            .argName("PORT")
+            .hasArg()
+            .required(false)
+            .desc("Port of a local Solr instance where you want to create the 
new core.")
+            .build(),
         SolrCLI.OPTION_SOLRURL,
         SolrCLI.OPTION_SOLRURL_DEPRECATED,
         SolrCLI.OPTION_ZKHOST,
-        SolrCLI.OPTION_ZKHOST_DEPRECATED);
+        SolrCLI.OPTION_ZKHOST_DEPRECATED,
+        SolrCLI.OPTION_VERBOSE_DEPRECATED);

Review Comment:
   Same here:
   ```suggestion
           SolrCLI.OPTION_ZKHOST_DEPRECATED,
           SolrCLI.OPTION_VERBOSE,
           SolrCLI.OPTION_VERBOSE_DEPRECATED);
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to