jadami10 opened a new pull request, #17450: URL: https://github.com/apache/pinot/pull/17450
This is a followup to #17248 since I don't have access to edit. This closes #16775. Similar to the original PR, this distinguishes servers into new enum states that are correctly displayed in the UI. I removed the part where we call `/health` on individual servers because 1. many users don't or shouldn't have access directly to servers from the browser 2. pushing the `/health` call through the controller is complicated and likely expensive for large clusters So for controllers/brokers/minions, you will only ever see `Dead` and `Healthy`. For servers you will see the other statuses. I tested this by running a local UI against one of our QA clusters, performing a rolling restart, and ensuring the server each transitioned through these expected states. -- 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]
