vrajat commented on PR #15346: URL: https://github.com/apache/pinot/pull/15346#issuecomment-2745032294
I re-read the code to see if the call to broker request can be added on instead of reverting the commit. The only other option I see is : * Add the function to authorize using broker request. * Remove the table check from `authorize(..., BrokerRequest ...)` e.g. https://github.com/apache/pinot/blob/master/pinot-broker/src/main/java/org/apache/pinot/broker/broker/ZkBasicAuthAccessControlFactory.java#L100 -- 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]
