Github user andytaylor commented on the pull request:
https://github.com/apache/activemq-artemis/pull/227#issuecomment-99430232
personally I wouldn't have used Interceptors. Its a level of indirection
that isn't really needed. I would just have used abstraction and catch the
packets in extension points. It's easier to read and understand from a coding
pov and we can be sure that performance isn't an issue or wont be an issue if
someone changes the code. Saying that, if no one else has any misgivings then
I'm happy for it to be merged.
---
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.
---
