gortiz opened a new pull request, #17576: URL: https://github.com/apache/pinot/pull/17576
Since the introduction of stage stats, pipeline breakers were not supported. The query doesn't fail, but the stats from the stages executed in the pipeline breaker weren't included. This is especially problematic when the pipeline breaker side includes warnings like `numGroupsLimitReached`, because not including the stageStats means we do not set that flag. This PR solves that issue. Now leaf operators that include a pipeline breaker return the pipeline breaker upstream stats in the same way we do in MailboxReceiveOperators. This change doesn't break compatibility because, by default, if the cluster runs different versions, we don't send stageStats. TODO: - [ ] Introduce a server config to disable this feature. -- 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]
