eolivelli commented on a change in pull request #9975:
URL: https://github.com/apache/pulsar/pull/9975#discussion_r598764623
##########
File path:
pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdBrokers.java
##########
@@ -146,6 +146,15 @@ void run() throws Exception {
}
+ @Parameters(commandDescription = "Get the version of the currently
connected broker")
+ private class PulsarVersion extends CliCommand {
+
+ @Override
+ void run() throws Exception {
+ System.out.println( getAdmin().brokers().getVersion());
Review comment:
I believe that we have a little usability problem here.
The user does not know which broker reported the version.
Proposals:
- return the address of the broker that reported the version (a BrokerInfo
structure ?)
- add a (optional) parameter to specify the id of the broker that we want to
query
In a Pulsar cluster usually you have the same version of the software, but
this may not be the case during upgrades or in case you have special versions
of Pulsar on some nodes.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
