BewareMyPower commented on PR #20848: URL: https://github.com/apache/pulsar/pull/20848#issuecomment-1644994368
I believe some other APIs are also affected by the behavior of these default methods, like `Topics#getPermissions`, `Namespaces#getPermissionOnSubscription`. It's also very confusing to see the methods added in https://github.com/apache/pulsar/pull/20496 don't accept any role argument. How could users know how to implement them without knowing any role? Is it beyond the scope of "authorization"? The thoughts above are out of the scope of this PR, this PR only aims at fixing the regression of deleting topics. -- 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]
