You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by cs...@apache.org on 2019/09/06 16:19:56 UTC

[activemq] branch master updated: AMQ-7300 - Remove unnecessary call to sequence updater as read is already volatile

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 65de327  AMQ-7300 - Remove unnecessary call to sequence updater as read is already volatile
65de327 is described below

commit 65de327369f24de008f857211b97d3c69283a932
Author: Christopher L. Shannon (cshannon) <ch...@gmail.com>
AuthorDate: Fri Sep 6 12:19:23 2019 -0400

    AMQ-7300 - Remove unnecessary call to sequence updater as read is already volatile
---
 .../src/main/java/org/apache/activemq/util/LongSequenceGenerator.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/activemq-client/src/main/java/org/apache/activemq/util/LongSequenceGenerator.java b/activemq-client/src/main/java/org/apache/activemq/util/LongSequenceGenerator.java
index f3590a9..0d7eb0b 100644
--- a/activemq-client/src/main/java/org/apache/activemq/util/LongSequenceGenerator.java
+++ b/activemq-client/src/main/java/org/apache/activemq/util/LongSequenceGenerator.java
@@ -16,7 +16,6 @@
  */
 package org.apache.activemq.util;
 
-import java.util.concurrent.atomic.AtomicLong;
 import java.util.concurrent.atomic.AtomicLongFieldUpdater;
 
 /**
@@ -35,7 +34,7 @@ public class LongSequenceGenerator {
     }
 
     public long getLastSequenceId() {
-        return SEQUENCE_UPDATER.get(this);
+        return lastSequenceId;
     }
 
     public void setLastSequenceId(long l) {