Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1507#discussion_r136985490
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ServerSessionImpl.java
---
@@ -348,7 +348,9 @@ public boolean removeConsumer(final long consumerID)
throws Exception {
}
protected void doClose(final boolean failed) throws Exception {
- callback.close(failed);
--- End diff --
callback is a final... I would rather fix why this became null.. this
wasn't supposed to be null.
---
