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 2021/02/20 09:54:00 UTC

[GitHub] [pulsar] 315157973 opened a new pull request #9657: Fix MaxUnackedMessagesPerConsumer cannot be changed dynamically and cannot be disabled

315157973 opened a new pull request #9657:
URL: https://github.com/apache/pulsar/pull/9657


   ### Motivation
   1. The `MaxUnackedMessagesPerConsumer` cannot be changed dynamically now. When the policy is modified, only the newly created consumer can use the new policy. The previously created consumer still uses the old policy.
   It is very difficult to restart the consumer in the online environment.
   2. This policy cannot be disabled.
   
   ### Modifications
   1. Change `int` to `Integer`, so that we can judge whether the namespace-level has a policy
   2. The `maxUnackedMessages` in the consumer is changed to non-final, and this value is also updated when the policy is updated
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [pulsar] codelipenghui merged pull request #9657: Fix MaxUnackedMessagesPerConsumer cannot be changed dynamically and cannot be disabled

Posted by GitBox <gi...@apache.org>.
codelipenghui merged pull request #9657:
URL: https://github.com/apache/pulsar/pull/9657


   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org