[jira] [Commented] (SOLR-9272) Auto resolve zkHost for bin/solr zk for running Solr

Previous Topic Next Topic
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view

[jira] [Commented] (SOLR-9272) Auto resolve zkHost for bin/solr zk for running Solr

JIRA jira@apache.org

    [ https://issues.apache.org/jira/browse/SOLR-9272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16088228#comment-16088228 ]

Amrit Sarkar commented on SOLR-9272:


{{StatusTool}} implements abstract method {{runImpl}} of SolrCli.ToolBase of return type {{void}}. In order to fetch {{cloud/Zookeeper}} we have to change the base abstract method return type and corresponding all its classes which implements it.

StatusTool and all other classes which extends ToolBase, {{echo}} the output and returns {{void}}. I would like to drive this to conclusion; hence seeking suggestion whether it is a good idea to change base abstract method's return type or it is too much change for too little convenience.

> Auto resolve zkHost for bin/solr zk for running Solr
> ----------------------------------------------------
>                 Key: SOLR-9272
>                 URL: https://issues.apache.org/jira/browse/SOLR-9272
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public)
>          Components: scripts and tools
>    Affects Versions: 6.2
>            Reporter: Jan Høydahl
>            Assignee: Jan Høydahl
>              Labels: newdev
> Spinoff from SOLR-9194:
> We can skip requiring {{-z}} for {{bin/solr zk}} for a Solr that is already running. We can optionally accept the {{-p}} parameter instead, and with that use StatusTool to fetch the {{cloud/ZooKeeper}} property from there. It's easier to remember solr port than zk string.
> Example:
> {noformat}
> bin/solr start -c -p 9090
> bin/solr zk ls / -p 9090
> {noformat}

This message was sent by Atlassian JIRA

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