(activemq) branch main updated: [#2275] Drop redundant sequence generator monitor in RegionBroker (#2283)

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

mattrpav pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/activemq.git


The following commit(s) were added to refs/heads/main by this push:
     new cd1dfa298c [#2275] Drop redundant sequence generator monitor in RegionBroker (#2283)
cd1dfa298c is described below

commit cd1dfa298c11dd614fdc16bad0b694639f185991
Author: Matt Pavlovich <ma...@apache.org>
AuthorDate: Wed Aug 5 17:49:03 2026 -0500

    [#2275] Drop redundant sequence generator monitor in RegionBroker (#2283)
---
 .../main/java/org/apache/activemq/broker/region/RegionBroker.java   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/activemq-broker/src/main/java/org/apache/activemq/broker/region/RegionBroker.java b/activemq-broker/src/main/java/org/apache/activemq/broker/region/RegionBroker.java
index 7d5de4c184..ed74493e15 100644
--- a/activemq-broker/src/main/java/org/apache/activemq/broker/region/RegionBroker.java
+++ b/activemq-broker/src/main/java/org/apache/activemq/broker/region/RegionBroker.java
@@ -865,9 +865,9 @@ public class RegionBroker extends EmptyBroker {
      */
     @Override
     public long getBrokerSequenceId() {
-        synchronized (sequenceGenerator) {
-            return sequenceGenerator.getNextSequenceId();
-        }
+        // LongSequenceGenerator is atomic; no monitor needed on this hot path
+        // (stamped on every send for every destination).
+        return sequenceGenerator.getNextSequenceId();
     }
 
     @Override


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@activemq.apache.org
For additional commands, e-mail: commits-help@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact