Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-artemis/pull/142#issuecomment-135845502
@howardgao I confused the class with AMQDestination under amq on the
protocolManager, that's not used anywhere.
I see that this is about temp destinations... although I would prefer that
code (for deletion) out of the ProtocolManager, since the ProtocolManager is
doing a lot more work than what is supposed to be doing...
I will merge your commit, although I'm looking to simplify some of these
function after we release 1.2.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
