gemmellr commented on code in PR #4183:
URL: https://github.com/apache/activemq-artemis/pull/4183#discussion_r1067119685
##########
artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/amq/AMQSession.java:
##########
@@ -392,7 +392,7 @@ public void send(final ProducerInfo producerInfo,
assert
clientId.toString().equals(this.connection.getState().getInfo().getClientId())
: "Session cached clientId must be the same of the connection";
originalCoreMsg.putStringProperty(MessageUtil.CONNECTION_ID_PROPERTY_NAME,
clientId);
-
+
originalCoreMsg.putStringProperty(org.apache.activemq.artemis.api.core.Message.PRODUCER_ID,
producerInfo.getProducerId().toString());
Review Comment:
Lets make things better instead of making it worse. Doing the same crap
thing 'because someone else did before' only ever leads in the wrong direction.
Field on the message object, e.g the way you did it for the AMQP messages?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
