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:
[email protected]


With regards,
Apache Git Services

Reply via email to