You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by lh...@apache.org on 2025/02/25 11:07:16 UTC
(pulsar) 02/03: [fix][broker] Fix BucketDelayedDeliveryTracker thread safety (#24014)
This is an automated email from the ASF dual-hosted git repository.
lhotari pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/pulsar.git
commit 4e39f705a9543975edc347f1b6d69224af559114
Author: 道君 <da...@apache.org>
AuthorDate: Tue Feb 25 14:51:19 2025 +0800
[fix][broker] Fix BucketDelayedDeliveryTracker thread safety (#24014)
(cherry picked from commit a83dd86756bd540c8cc0991afcd3d2ba333603b7)
---
.../pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java
index c23bc17c303..65a92dcd8b9 100644
--- a/pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java
+++ b/pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java
@@ -754,7 +754,7 @@ public class BucketDelayedDeliveryTracker extends AbstractDelayedDeliveryTracker
.orElse(false);
}
- public boolean containsMessage(long ledgerId, long entryId) {
+ public synchronized boolean containsMessage(long ledgerId, long entryId) {
if (lastMutableBucket.containsMessage(ledgerId, entryId)) {
return true;
}