You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2018/02/21 00:47:41 UTC

[GitHub] nkurihar commented on issue #1255: Add namespace policy for max clients

nkurihar commented on issue #1255: Add namespace policy for max clients
URL: https://github.com/apache/incubator-pulsar/pull/1255#issuecomment-367174429
 
 
   @merlimat 
   > If having unlimited producers on a topic is bad for the broker, we shouldn't allow a tenant to override the setting and allow it.
   
   I agree it.
   
   Then, how about restricting this command to only superUser like DispatchRate?
   https://github.com/apache/incubator-pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/NamespacesBase.java#L587

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services