Jackie-Jiang commented on issue #9023: URL: https://github.com/apache/pinot/issues/9023#issuecomment-1191866665
Checked other systems, and seems health check is mostly for liveness instead of readiness. Since we already use it as the readiness check, we can leave it as is for backward compatibility. Based on the requirement, I'd suggest making the api more explicit on the purpose: - `/health/liveness` for liveness - `/health/readiness` for readiness I feel using `instance` and `service` is not as clear -- 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]
