codelipenghui commented on issue #5936: Add maxUnackedMessagesPerSubscription and maxUnackedMessagesPerConsumer on namespaces policies URL: https://github.com/apache/pulsar/pull/5936#issuecomment-574969960 @sijie I took a look, maybe not solve my previous comment. I think @wolfstudy can add a unit test to ensure the dispatcher can handle the policy update. because your current unit test works on zookeeper (write policy data to zookeeper and get policy data from zookeeper), this does not ensure that the dispatcher can work well with policy update.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
