[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: RE: [amqp-bindmap] Groups - Advanced Message Queuing Protocol (AMQP) JMS Mapping Version 1.0 WD07 uploaded
Looks good. A couple of comments from me: 3.2.1 JMS Headers: JMSMessageID / 3.2.1.1 JMSMessageID and JMSCorrelationID Handling For a request/response pattern where the response queue is fixed, I expect to able to send a message a request queue, then create a consumer on the response
queue with a selector that filters for my response e.g. createConsumer(responseQueue, String.format(“JMSCorrelationID = ‘%s’”, requestMessage.get JMSMessageID()).
If I, as a JMS client library vendor, were to implement the JMS mapping and elect to use message-ids of type message-id-uuid, then I think the application calling
get JMSMessageID() would see “ID:AMQP_UUID:<string representation of UUID>”. This would mean the JMS selector produced by the application executing the line above would include the type encoding prefix “ID:AMQP_UUID”. As I assume the type encoding prefixes
are private, the selector would then fail to match on the server peer. What is the rationale in 3.2.1 reading “SHOULD use message-id-string”? 3.3.2 Properties Section: User Id What is behaviour is required if the user-id is not legal UTF-8? 5.3 Temporary Destinations The text ought to mention the use of terminus capabilities temporary-queue/temporary topic. General There are a number of places where we talk about the need to throw an exception, but don’t actually define what it should be. From: amqp-bindmap@lists.oasis-open.org [mailto:amqp-bindmap@lists.oasis-open.org]
On Behalf Of Robert Gemmell
This email is confidential and subject to important disclaimers and conditions including on offers for the purchase or sale of securities, accuracy and completeness of information, viruses, confidentiality, legal privilege, and legal entity disclaimers, available at http://www.jpmorgan.com/pages/disclosures/email |
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]