[GitHub] activemq-artemis issue #2007: ARTEMIS-1799 - Add a NotificationActiveMQServe...

2018-04-10 Thread cshannon
Github user cshannon commented on the issue: https://github.com/apache/activemq-artemis/pull/2007 @clebertsuconic - thanks for merging ---

[GitHub] activemq-artemis issue #2007: ARTEMIS-1799 - Add a NotificationActiveMQServe...

2018-04-10 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/2007 @cshannon I see. .thanks for the clarification. merging it. ---

[GitHub] activemq-artemis issue #2007: ARTEMIS-1799 - Add a NotificationActiveMQServe...

2018-04-10 Thread cshannon
Github user cshannon commented on the issue: https://github.com/apache/activemq-artemis/pull/2007 I don't understand the confusion. That call just calls off to any registered plugins if they exist. This commit is a brand new plugin that implements the afterDeliver call and fires