Yuyaoo opened a new pull request, #15346: URL: https://github.com/apache/pinot/pull/15346
PR apache/pinot#15240 introduced a backwards-incompatible change that has broken our build. Specifically, it removed the use of `accessControl.authorize` with the `brokerRequest` ([previous usage](https://github.com/apache/pinot/pull/15240/files#diff-06e2bf5ea89ac7b1f67fa90f3d3e9bbb8c61cae03c20093fd18bdc977f949134L381)). Our system relies heavily on this `brokerRequest` to perform some query analysis and parsing, making this removal problematic This PR temporarily revert the change to restore compatibility while we explore a long-term solution, for example exposing a separate function to pass the `brokerRequest`. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
