Repository: qpid-jms Updated Branches: refs/heads/master 803400254 -> 459893d36
Add more to the TODO comment. Project: http://git-wip-us.apache.org/repos/asf/qpid-jms/repo Commit: http://git-wip-us.apache.org/repos/asf/qpid-jms/commit/459893d3 Tree: http://git-wip-us.apache.org/repos/asf/qpid-jms/tree/459893d3 Diff: http://git-wip-us.apache.org/repos/asf/qpid-jms/diff/459893d3 Branch: refs/heads/master Commit: 459893d36926c73dae7627046c4f70740ab83cc5 Parents: 8034002 Author: Timothy Bish <[email protected]> Authored: Wed Oct 8 18:24:55 2014 -0400 Committer: Timothy Bish <[email protected]> Committed: Wed Oct 8 18:24:55 2014 -0400 ---------------------------------------------------------------------- .../src/main/java/org/apache/qpid/jms/message/JmsMessage.java | 4 ++++ 1 file changed, 4 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/459893d3/qpid-jms-client/src/main/java/org/apache/qpid/jms/message/JmsMessage.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/main/java/org/apache/qpid/jms/message/JmsMessage.java b/qpid-jms-client/src/main/java/org/apache/qpid/jms/message/JmsMessage.java index 1a2450e..33ed19c 100644 --- a/qpid-jms-client/src/main/java/org/apache/qpid/jms/message/JmsMessage.java +++ b/qpid-jms-client/src/main/java/org/apache/qpid/jms/message/JmsMessage.java @@ -244,6 +244,10 @@ public class JmsMessage implements javax.jms.Message { //TODO: Handle any other relevant JMSX properties. //TODO: Possibly push this to the facade or interceptors? + // It makes sense to chain the call to clear through the JmsMessagePropertyIntercepor + // which would in turn call the clear on the facade which could then allow its own + // intercepter to clear any managed JMS_AMQP properties before the facade does it's + // final clear on the message level properties. //Clear property-defined values that are treated as facade-managed, such as GroupSequence. facade.clearGroupSequence(); } --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
