gemmellr commented on code in PR #5477:
URL: https://github.com/apache/activemq-artemis/pull/5477#discussion_r1934281873
##########
artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/ProtonProtocolManager.java:
##########
@@ -374,8 +371,8 @@ public String invokeIncoming(Message message,
ActiveMQProtonRemotingConnection c
// For tunneled messages we need to check the type as our interceptor
only cares about
// AMQP message right now so there's not notification point for other
types that cross
if (incomingInterceptors != null && !incomingInterceptors.isEmpty()) {
- if (message instanceof AMQPMessage) {
- return super.invokeInterceptors(this.incomingInterceptors,
(AMQPMessage) message, connection);
+ if (message instanceof AMQPMessage pMessage) {
+ return super.invokeInterceptors(this.incomingInterceptors,
pMessage, connection);
Review Comment:
```suggestion
if (message instanceof AMQPMessage amqpMessage) {
return super.invokeInterceptors(this.incomingInterceptors,
amqpMessage, connection);
```
same below
--
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
