janhoy commented on code in PR #2712: URL: https://github.com/apache/solr/pull/2712#discussion_r1759757260
########## solr/core/src/java/org/apache/solr/cli/StatusTool.java: ########## @@ -65,37 +74,119 @@ public String getName() { .desc("Wait up to the specified number of seconds to see Solr running.") .build(); + public static final Option OPTION_PORT = + Option.builder("p") + .longOpt("port") + .argName("PORT") + .required(false) + .hasArg() + .desc("Port on localhost to check status for") + .build(); + + public static final Option OPTION_SHORT = + Option.builder() + .longOpt("short") + .argName("SHORT") + .required(false) + .desc("Short format. Prints one URL per line for running instances") + .build(); + @Override public List<Option> getOptions() { - return List.of( - // The solr-url option is not exposed to the end user, and is - // created by the bin/solr script and passed into this command directly, - // therefore we don't use the SolrCLI.OPTION_SOLRURL. - Option.builder() - .argName("URL") - .longOpt("solr-url") - .hasArg() - .required(false) - .desc("Property set by calling scripts, not meant for user configuration.") - .build(), - OPTION_MAXWAITSECS); + return List.of(OPTION_SOLRURL, OPTION_MAXWAITSECS, OPTION_PORT, OPTION_SHORT); } @Override public void runImpl(CommandLine cli) throws Exception { - // Override the default help behaviour to put out a customized message that only list user - // settable Options. - if ((cli.getOptions().length == 0 && cli.getArgs().length == 0) - || cli.hasOption("h") - || cli.hasOption("help")) { - final Options options = new Options(); - options.addOption(OPTION_MAXWAITSECS); - new HelpFormatter().printHelp("status", options); - return; + String solrUrl = cli.getOptionValue(OPTION_SOLRURL); Review Comment: Probably not, as it has never been advertised to end users? -- 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