Skip to content

[#2418] Improve performance of IntSequenceGenerator - #2408

Merged
cshannon merged 1 commit into
apache:mainfrom
cshannon:int-sequence-generator-fix
Aug 6, 2026
Merged

[#2418] Improve performance of IntSequenceGenerator#2408
cshannon merged 1 commit into
apache:mainfrom
cshannon:int-sequence-generator-fix

Conversation

@cshannon

@cshannon cshannon commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

This adopts the same approach as LongSequenceGenerator and uses an Atomic field updater to avoid using synchronized.

The sequence generator is used by ResponseCorrelator so the existing tests will catch any errors.

See https://issues.apache.org/jira/browse/AMQ-7300

This adopts the same approach as LongSequenceGenerator and uses an
Atomic field updater to avoid using synchronized.

The sequence generator is used by ResponseCorrelator so the existing
tests will catch any errors.

See https://issues.apache.org/jira/browse/AMQ-7300
@cshannon
cshannon force-pushed the int-sequence-generator-fix branch from 7737661 to 074f325 Compare August 5, 2026 23:16

@mattrpav mattrpav left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find! LGTM

@mattrpav mattrpav changed the title Improve performance of IntSequenceGenerator [#2418] Improve performance of IntSequenceGenerator Aug 6, 2026
@cshannon
cshannon merged commit 9c3add1 into apache:main Aug 6, 2026
10 checks passed
@cshannon
cshannon deleted the int-sequence-generator-fix branch August 6, 2026 13:02
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Apache ActiveMQ v6.4.0 Aug 6, 2026
cshannon added a commit that referenced this pull request Aug 6, 2026
This adopts the same approach as LongSequenceGenerator and uses an
Atomic field updater to avoid using synchronized.

The sequence generator is used by ResponseCorrelator so the existing
tests will catch any errors.

See https://issues.apache.org/jira/browse/AMQ-7300

(cherry picked from commit 9c3add1)
cshannon added a commit that referenced this pull request Aug 6, 2026
This adopts the same approach as LongSequenceGenerator and uses an
Atomic field updater to avoid using synchronized.

The sequence generator is used by ResponseCorrelator so the existing
tests will catch any errors.

See https://issues.apache.org/jira/browse/AMQ-7300

(cherry picked from commit 9c3add1)
cshannon added a commit that referenced this pull request Aug 6, 2026
This adopts the same approach as LongSequenceGenerator and uses an
Atomic field updater to avoid using synchronized.

The sequence generator is used by ResponseCorrelator so the existing
tests will catch any errors.

See https://issues.apache.org/jira/browse/AMQ-7300

(cherry picked from commit 9c3add1)
@cshannon cshannon linked an issue Aug 6, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Improve performance of IntSequenceGenerator

2 participants