yuruguo edited a comment on pull request #13463:
URL: https://github.com/apache/pulsar/pull/13463#issuecomment-1000802912


   @massakam Thanks for your answers
   > As far as internalSetMaxProducersPerTopic and 
internalSetMaxConsumersPerTopic are concerned, it was my colleague who added 
these features, and initially only superusers were able to perform these 
operations.
   
   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 internalSetMaxSubscriptionsPerTopic should only be performed by 
the superuser. 
   
   I agree with it.
   
   **Finally**, I am not opposed to revert the PR(#13157), but maybe it's not a 
good way to revert it directly or completely, the reasons are as follows:
   - there are some corrections to annotations that are not entirely related to 
the modified authz of namespace policies
   - it is possible for tenant administrator not only super user to operate 
`internalSetInactiveTopic` / `internalSetDelayedDelivery`
   
   Therefore, it may be a better solution to revert purposeful, such as 
creating a special PR


-- 
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]


Reply via email to