sascha-coenen 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_r344305509
 
 

 ##########
 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:
   readiness is a better term IMHO because it is an official term for health 
checking that also has well defined semantics:
   
   
https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/
   
   More and more apps are exposing health check endpoints to allow for easy 
integration into orhestration solutions like Kubernetes

----------------------------------------------------------------
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]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to