Jackie-Jiang commented on issue #9023: URL: https://github.com/apache/pinot/issues/9023#issuecomment-1189679015
> One concern I have with a more complex API is most load balancers only allow configuring the path and then look at the status code of the response. > > So having both /health/instance and /health/services like in the proposal makes most sense to me. @jadami10 What if we return different status code for different status? Current API only supports `200` and `503`, which cannot represent all the available status. One problem is that different client might have different logic on parsing the pinot service status, and fixing the return type can reduce the flexibility. We don't want to add an extra new API for each different status check logic. -- 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]
