janhoy commented on code in PR #5037:
URL: https://github.com/apache/solr/pull/5037#discussion_r4205513234


##########
solr/core/src/java/org/apache/solr/cli/ApiTool.java:
##########
@@ -45,6 +59,21 @@ public class ApiTool extends ToolBase {
   /** Parameters for the api command, independent of the command line parser. 
*/
   record ApiParams(String getUrl, String credentials) {}
 
+  // --- picocli fields ---
+
+  @picocli.CommandLine.Option(
+      names = {"-s", "--solr-url"},
+      required = true,
+      paramLabel = "URL",
+      description = "Send a GET request to a Solr API endpoint.")

Review Comment:
   This describes the command, not the value. "Full URL of the Solr API 
endpoint to GET, e.g. http://localhost:8983/api/cluster"; reads better on the 
generated ref page.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to