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

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

clintropolis 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_r356331125
 
 

 ##########
 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:
   I think 'readiness' is the correct term for this, and putting it at `/druid/broker/v1/readiness` seems acceptable for me since there is precedent with historicals. A universal `/readiness` I think might take a bigger refactor since there isn't really a universal concept of readiness across all node types, or any sort of health style interface that all node types implement, but is worth considering in the future.

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