tuteng commented on a change in pull request #9975:
URL: https://github.com/apache/pulsar/pull/9975#discussion_r599153778
##########
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:
Thanks, I agree with you, but I think the current version of the broker
can be queried explicitly using the `pulsar-admin brokers version` command, for
example, the `pulsar-admin brokers version` command will use the
`webServiceUrl` configuration in the `client.conf` configuration file, and of
course, the `webServiceUrl` parameter can be used to specify the address of the
broker to be queried by the command `pulsar-admin --admin-url`, In a pulsar
cluster upgrade, multiple versions of the broker will exist at the same time,
and with the `--admin-url` parameter, we can explicitly get the version of the
specified broker
--
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]
