janhoy commented on code in PR #5190: URL: https://github.com/apache/solr/pull/5190#discussion_r4351219154
########## solr/solr-ref-guide/modules/deployment-guide/pages/solr-control-script-reference.adoc: ########## @@ -29,8 +29,8 @@ More examples of SolrCLI in use are available throughout this Guide, particularl ==== This page documents the `bin/solr` script as it behaves by default. -An experimental re-implementation of the command line interface, built on picocli, is also included in this release. -It is disabled by default, and the options it accepts differ in places from those documented here. +A re-implementation of the command line interface, built on picocli, is also included in this release. +It is enabled by default, and the options it accepts differ in places from those documented here. See xref:cli/index.adoc[] if you would like to try it. ==== Review Comment: I think this NOTE shuold be replaced with a short note about commons-cli being available as an escape hatch. The link to `xref:cli/index.adoc[]` should instead be a new paragraph `== Man pages` or similar, telling that each command has its own documentation page and the link. I wonder if perhpas most of the remainder of this adoc page can be deleted since it is manually maintained and largely redundant with the new man pages? There are some general sections near the bottom that could perhaps stay. -- 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]
