[GitHub] merlimat commented on issue #1654: Enable dispatch-throttling for non backlog consumers by default.

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

merlimat commented on issue #1654: Enable dispatch-throttling for non backlog consumers by default.
URL: https://github.com/apache/incubator-pulsar/pull/1654#issuecomment-384783167
 
 
   >  Default if broker throttles dispatching of backlogged subscription then isn't it create more and more backlog.? So, default broker should not throttle backlogged cursor and in case if it requires then it's a dynamic configuration which can be applied at runtime for some special case.
   
   Yes, that was the reason we had it this way. Though, I agree with @srkukarni  that it's very unintuitive from a user perspective, because "I did set the dispatch rate and I'm not being throttled". 
   
   A better approach could be to have "throttleOnlyBacklogged" flag to be at the namespace level. So that we could do : 
   ```shell
   pulsar-admin namespaces set-dispatch-rate -r 1000 $MY_NAMESPACE --only-when-backlogged
   ```

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