Skip to content

ARTEMIS-6286 Use concurrent map for MQTTStateManager instances - #6757

Merged
jbertram merged 1 commit into
apache:mainfrom
amarkevich:MQTTStateManager
Oct 6, 2026
Merged

jbertram merged 1 commit into
apache:mainfrom
amarkevich:MQTTStateManager

Conversation

@amarkevich

Copy link
Copy Markdown
Contributor

No description provided.

@jbertram

jbertram commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

Is there a particular motivation for this change? All I can see is a minor reduction in code complexity.

@amarkevich

Copy link
Copy Markdown
Contributor Author

Is there a particular motivation for this change? All I can see is a minor reduction in code complexity.

yes - align multi-thread logic to use synchronized collections

@jbertram

jbertram commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

I opened https://issues.apache.org/jira/browse/ARTEMIS-6286 for this. You should reference it in your commit message along with a shorter description to adhere to our standards.

@amarkevich amarkevich changed the title MQTTStateManager: drop synchronized in favor of ConcurrentHashMap ARTEMIS-6286 Use concurrent map for MQTTStateManager instances Oct 6, 2026
@jbertram
jbertram merged commit 884d99d into apache:main Oct 6, 2026
6 checks passed
@amarkevich
amarkevich deleted the MQTTStateManager branch October 6, 2026 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants