lovelle opened a new pull request #5793: Fixes some minor thread safety violations URL: https://github.com/apache/pulsar/pull/5793 *Motivation* Reading latest changes through pulsar code I found some minor thread safety violations. *Modifications* - Add synchronized method to poll `pendingBatchReceives from a non-synchronized method. - Protects hasPendingBatchReceive method with synchronized. - When possible, prefer using final fields. fixed for initialStartMessageId and startMessageRollbackDurationInSec fields. - Fix issue accessing timeout field within inner lock on UnAckedMessageTracker.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
