xiangfu0 opened a new pull request, #16590: URL: https://github.com/apache/pinot/pull/16590
Context - Brokers used to wait on IdealState/ExternalView convergence before reporting GOOD. - When a broker is marked down in ZooKeeper/Helix, we want service status to reflect lifecycle immediately without blocking on heavy IS/EV scans. Change - In BaseBrokerStarter.registerServiceStatusHandler, register LifecycleServiceStatusCallback before Helix-based matchers so lifecycle transitions (STARTING/SHUTTING_DOWN) are reported promptly. - Leave IdealState/CurrentState and IdealState/ExternalView matchers intact for readiness, but they no longer delay lifecycle-driven fail-fast. Impact - Faster and more accurate status when a broker is disabled or marked down. - No additional work on IdealState/ExternalView during shutdown/startup. Testing - Compiled pinot-broker with dependencies. - Manually verified callback order in status description during startup/shutdown. -- 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]
