adoroszlai commented on code in PR #6433:
URL: https://github.com/apache/ozone/pull/6433#discussion_r1537234190
##########
hadoop-hdds/tools/src/main/java/org/apache/hadoop/hdds/scm/cli/ContainerBalancerStopSubcommand.java:
##########
@@ -34,6 +34,7 @@
public class ContainerBalancerStopSubcommand extends ScmSubcommand {
@Override
public void execute(ScmClient scmClient) throws IOException {
+ System.out.println("ContainerBalancer is in stopping state. Waiting for
current iteration to finish");
scmClient.stopContainerBalancer();
System.out.println("Stopping ContainerBalancer...");
Review Comment:
`...` indicates the command is doing something (waiting for the server). I
think it should appear before sending the stop command.
```
ContainerBalancer is in stopping state. Waiting for current iteration to
finish...
ContainerBalancer stopped.
```
or even (printed in two steps):
```
Stopping ContainerBalancer... done.
```
--
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]
