3c058e98f1
This commit fixes the deadlock described on ARTEMIS-3622 by moving the synchronization "up" a level from the MQTTSession to the MQTTConnectionManager. It also eliminates the synchronization on the MQTTSessionState in the MQTTConnectionManager because it's no longer needed. This change should not only eliminate the deadlock, but improve performance relatively as well. There is no test associated with this commit as I wasn't able to reproduce the deadlock with any kind of straight-forward test. There was a test linked on the Jira, but it involved intrusive and fragile scaffolding and wasn't ultimately tenable. That said, I did test this fix with that test and it was successful. In any case, I think static analysis should be sufficient here as the changes are pretty straight-forward. |
||
|---|---|---|
| .. | ||
| artemis-amqp-protocol | ||
| artemis-hornetq-protocol | ||
| artemis-hqclient-protocol | ||
| artemis-jakarta-openwire-protocol | ||
| artemis-mqtt-protocol | ||
| artemis-openwire-protocol | ||
| artemis-stomp-protocol | ||
| pom.xml | ||