[GitHub] [pulsar] michaeljmarshall commented on pull request #13463: Revert "[Authorization] Converge authz of ns policies from super-user to tenant-administrator (#13157)"

[GitBox <gi...@apache.org>]

michaeljmarshall commented on pull request #13463:
URL: https://github.com/apache/pulsar/pull/13463#issuecomment-1006093023


   I lean towards reverting the whole commit and then creating new PRs to add the right functionality. However, I don't have a strong opinion here.
   
   > Maybe we can find the reason why the access permission was changed from super user to tenant administrator based on the submission history.
   
   I think @massakam's point is that #13157 decreased the necessary permission level and that it has problematic consequences for things like tenant rate limits. I agree that we should leave the rate limit permissions at the superuser level for now until we make it possible to set an upper limit for a given tenant.
   
   We should figure this out before 2.10.0 gets released.


-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org