Nikita-Shupletsov commented on PR #1247:
URL: https://github.com/apache/activemq/pull/1247#issuecomment-2219181991

   > What is the use case for this?
   
   you mean the use case of having a DestinationFilter after 
SelectorAwareVirtualTopicInterceptor?
   
   I my case I add an interceptor to all topics. and that interceptor coexists 
with the rest of interceptors. and some topics can be vistual topics. it's 
technically possible on the plugin side to check if there is a virtual topic 
interceptor and not add my interceptor. But I checked other interceptors([this 
one](https://github.com/apache/activemq/blob/main/activemq-broker/src/main/java/org/apache/activemq/broker/region/virtual/MappedQueueFilter.java#L68)
 for example) and they handle this situation correctly, so I thought it would 
be a positive change in general. In case people decide to do something similar 
in the future, they will not need to work around the edge case.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org
For additional commands, e-mail: gitbox-h...@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact


Reply via email to