jsr343-experts@jms-spec.java.net

[jsr343-experts] Re: (JMS_SPEC-112) add Message.getJMSDeliveryCount()

From: John D. Ament <john.d.ament_at_gmail.com>
Date: Wed, 9 Jan 2013 13:36:28 -0500

Agreed to leaving it as is, no benefit from having a specific property
being given a dedicated "getter."


On Wed, Jan 9, 2013 at 1:34 PM, Nigel Deakin <nigel.deakin_at_oracle.com>wrote:

> "axel_podehl" has logged this JIRA issue:
>
> -------- Original Message --------
> (JMS_SPEC-112) add Message.getJMSDeliveryCount()
> http://java.net/jira/browse/**JMS_SPEC-112<http://java.net/jira/browse/JMS_SPEC-112>
>
> Since a JMSXDeliveryCount will be part of the JMS 2.0 requirement, also
> the Message interface should reflect this:
>
> public long getJMSDeliveryCount() throws JMSException
>
> ------------------------------**------------------------------**
> ------------------------------**------------------------
>
> JMS defines two different type of information which can be associated with
> a message (apart from the body): message headers and message properties.
> Message headers are accessed using specific javabean-like methods such as
> getJMSPriority whereas message properties are accessed using generic
> methods such as getIntProperty(name).
>
> JMSXDeliveryCount was defined (though not made mandatory) in JMS 1.1 and
> is a message property, not a message header, which is why it is accessed
> using setIntProperty and getIntProperty rather than more specific methods.
> In JMS 2.0 all we are doing is making this existing property mandatory.
>
> I can see a case for defining specific methods to access the JMS-defined
> properties (those starting with JMSX), or even just the mandatory one(s),
> but there is potential confusion in providing two ways to access the same
> value. And since JMSXDeliveryCount was defined in JMS 1.1 it has to remain
> a property.
>
> So I propose we leave things as they are. However I'll leave this issue
> open for now to allow for other comments either via the issue or via the
> EG/user alias.
>
>
>
> Nigel
>