Skip to content

Comments

fix ConcurrentModificationException in async command sending#13626

Open
Gerrygames wants to merge 2 commits intoPaperMC:mainfrom
Gerrygames:fix/async-command-sending
Open

fix ConcurrentModificationException in async command sending#13626
Gerrygames wants to merge 2 commits intoPaperMC:mainfrom
Gerrygames:fix/async-command-sending

Conversation

@Gerrygames
Copy link
Contributor

fixes ConcurrentModificationException in async command sending (issue #11101)
the CME was caused by map.put(this.dispatcher.getRoot(), rootCommandNode); iterating the roots children asynchronously for hash code calculation. this is fixed by copying the whole root node.

different approach to #13233 without locking/synchronization

@Warriorrrr Warriorrrr added the type: bug Something doesn't work as it was intended to. label Feb 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug Something doesn't work as it was intended to.

Projects

Status: Awaiting review

Development

Successfully merging this pull request may close these issues.

ConcurrentModificationException caused by async command sending patch

2 participants