yuruguo commented on pull request #13463: URL: https://github.com/apache/pulsar/pull/13463#issuecomment-1001103013
@massakam Thanks for your meticulous answer 👍 > 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 by a special PR, what do @eolivelli @315157973 @michaeljmarshall think? -- 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]
