GitHub user tabish121 opened a pull request:
https://github.com/apache/activemq-artemis/pull/2284
ARTEMIS-2067 Clean up some code in the AMQP protocol handling paths
Cleans up some of the code on the proton event handler, most noteable:
1. Fix IOCallback creation on each outbound send, use single instance
as the handler only ever does a flush and has no attached state.
2. Fix redundent locking and unlocking of connection lock on the event
path that already ensures that lock is held.
3. Set presettle state on the server sender at attach as it cannot
change afterwards so checking on every message is not needed.
4. Improve buffer type checking on receive to reduce amount of work
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tabish121/activemq-artemis proton-cleanups
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/activemq-artemis/pull/2284.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 #2284
----
commit 4349bd16d406dd883b077ba3339bd89a70d4acd0
Author: Timothy Bish <tabish121@...>
Date: 2018-08-30T18:32:01Z
ARTEMIS-2067 Clean up some code in the AMQP protocol handling paths
Cleans up some of the code on the proton event handler, most noteable:
1. Fix IOCallback creation on each outbound send, use single instance
as the handler only ever does a flush and has no attached state.
2. Fix redundent locking and unlocking of connection lock on the event
path that already ensures that lock is held.
3. Set presettle state on the server sender at attach as it cannot
change afterwards so checking on every message is not needed.
4. Improve buffer type checking on receive to reduce amount of work
----
---
