[GitHub] [incubator-druid] nishantmonu51 commented on a change in pull request #8841: add readiness endpoints to processes having initialization delays

[GitBox <gi...@apache.org>]

nishantmonu51 commented on a change in pull request #8841: add readiness endpoints to processes having initialization delays
URL: https://github.com/apache/incubator-druid/pull/8841#discussion_r345279741
 
 

 ##########
 File path: docs/operations/api-reference.md
 ##########
 @@ -752,7 +752,11 @@ Returns segment information lists including server locations for the given datas
 
 * `/druid/broker/v1/loadstatus`
 
-Returns a flag indicating if the Broker knows about all segments in Zookeeper. This can be used to know when a Broker process is ready to be queried after a restart.
+Returns a flag indicating if the Broker knows about all segments in the cluster. This can be used to know when a Broker process is ready to be queried after a restart.
+
+* `/druid/broker/v1/readiness`
 
 Review comment:
   agree with @sascha-coenen readiness is a commonly used term for health checks. 
   One change though would be to have a generic endpoint */readiness* instead of node type specific endpoints. 
   would suggest removing /druid/<node_type>/v1 prefix. 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org