GitHub user mtaylor opened a pull request:
https://github.com/apache/activemq-artemis/pull/2279
ARTEMIS-2055 Lock LM on PacketHandler on clear
@clebertsuconic The original fix can still result in the same issue it's
just less frequent. I've added a locking approach here. The perf impact is
minimal there's only contention during broker shutdown. The original test I
added uses ByteMan to inject a wait/signal on to get the threads in the correct
position of the race. However, adding this lock causes the test to hang, as
the lock is doing what it's supposed to. Since the test just hangs, I've
removed it unless you can come up with a better suggestion?
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mtaylor/activemq-artemis ARTEMIS-2055-LOCK
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/activemq-artemis/pull/2279.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2279
----
commit 26762303465b0528f2c0bcd0f32d62f1167eb4d4
Author: Martyn Taylor <mtaylor@...>
Date: 2018-08-27T14:05:20Z
ARTEMIS-2055 Lock LM on PacketHandler on clear
----
---
