[GitHub] saandrews commented on issue #402: Rate limiting in the dispatcher

[gi...@git.apache.org]

saandrews commented on issue #402: Rate limiting in the dispatcher
URL: https://github.com/apache/incubator-pulsar/issues/402#issuecomment-317114449
 
 
   I had the same point, but the broker resource related load balancing is
   done by load manager. So, ideally if broker becomes busy, load manager
   should shed load to bring down its load.
   
   On Fri, Jul 21, 2017 at 1:42 PM, Matteo Merli <no...@github.com>
   wrote:
   
   > Yes, actually earlier in the PIP I have defined Broker level configuration
   > rather Cluster level configuration but team decided to go with Cluster
   > level configuration to make sure if namespace load by broker1 or broker2,
   > both the brokers should throttle with similar rate to have transparency of
   > rate-limiting.
   >
   > That doesn't change the fact that the limit is per-topic.
   >
   > I was proposing to have the limit per-broker, irrespective of whether
   > there's a per-topic limit or not and whether the limit is configured
   > cluster-wise or at individual brokers level.
   >
   > That would be to ensure the broker resources are not exceeded. When that
   > happens, delivery will be slowed down
   >
   > ?
   > You are receiving this because you were mentioned.
   > Reply to this email directly, view it on GitHub
   > <https://github.com/apache/incubator-pulsar/issues/402#issuecomment-317107464>,
   > or mute the thread
   > <https://github.com/notifications/unsubscribe-auth/ATk4LK-sk22Q92CPKKASzyulJhZOXoa7ks5sQQ0hgaJpZM4NYcrz>
   > .
   >
   
 
----------------------------------------------------------------
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