suvodeep-pyne opened a new pull request, #17580: URL: https://github.com/apache/pinot/pull/17580
## Summary - Add `TenantTagReadinessCallback` to verify broker has valid tenant tags (not just `broker_untagged`) - Add `RoutingReadinessCallback` to verify routing exists for all assigned tables before reporting healthy - Register both callbacks in `BaseBrokerStarter.registerServiceStatusHandler()` with documented ordering dependency ## Background This addresses a production issue where queries could be routed to brokers that were not yet ready to serve traffic, causing query failures. The root cause is that the existing health check only validates Helix state convergence but doesn't verify: 1. The broker has valid tenant tags assigned 2. Routing tables are built for assigned tables ## Test plan - [x] Unit tests for `TenantTagReadinessCallback` (4 tests) - [x] Unit tests for `RoutingReadinessCallback` (5 tests) - [ ] Manual testing in dev cluster -- 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]
