fx19880617 commented on a change in pull request #6134:
URL: https://github.com/apache/incubator-pinot/pull/6134#discussion_r509930439
##########
File path:
pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StartServiceManagerCommand.java
##########
@@ -69,6 +69,8 @@
private String _clusterName = DEFAULT_CLUSTER_NAME;
@Option(name = "-port", required = true, metaVar = "<int>", usage = "Pinot
service manager admin port, -1 means disable, 0 means a random available port.")
private int _port;
+ @Option(name = "-healthCheckAllComponents", metaVar = "<boolean>", usage =
"Pinot service manager health check returns the all components health check.
The health check returns OK when all the components are returning OK.")
Review comment:
make sense, adding an option of healthcheck services to be included in
the `/health` call.
----------------------------------------------------------------
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
