Properties set on messages should override those on publisher about rabbitmq-jms-client HOT 3 CLOSED

Well, according to the JMS specification, the current behavior is correct, e.g. for JMSPriority:

When a message is sent, this field is ignored. After completion of the send, it holds the value specified by the method sending the message.

And the Javadoc:

This method is for use by JMS providers only to set this field when a message is sent. This message cannot be used by clients to configure the priority level of the message.

Same thing for expiration and delivery mode.

I suggest we keep this behavior as the default one, but provide a flag to switch to the non-compliant but more natural behavior.

cc @michaelklishin @garyrussell @damonsutherland

from rabbitmq-jms-client.

I concur; I would expect the hierarchy to be

• method argument (in overloaded send())
  • existing message properties
    • producer defaults

However, it's tough to make a behavior change like that in a maintenance release.

We (Spring) typically handle cases like that with a boolean that retains the previous behavior in the current release and switches to the new behavior in the next release.

That avoids breaking existing apps that are leveraging the current behavior.

from rabbitmq-jms-client.

We can do it in 2.0 or introduce a swappable strategy for property merging.

from rabbitmq-jms-client.