You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@activemq.apache.org by "Andriy Hnativ (JIRA)" <ji...@apache.org> on 2010/07/20 16:58:51 UTC

[jira] Created: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
-----------------------------------------------------------------------------------------

                 Key: AMQ-2840
                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
             Project: ActiveMQ
          Issue Type: Bug
          Components: Broker
    Affects Versions: 5.4.0
            Reporter: Andriy Hnativ
             Fix For: 5.4.0


Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Gary Tully (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gary Tully resolved AMQ-2840.
-----------------------------

    Fix Version/s: 5.4.1
                       (was: 5.4.0)
       Resolution: Fixed

reverted the non jms spec compliant part of the patch in r987090

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Rick McGuire (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61713#action_61713 ] 

Rick McGuire commented on AMQ-2840:
-----------------------------------

The axis2 release we're waiting on for the Geronimo release has finally gone to a vote.  We're going to be needed the 5.4.1 release shortly too.  The snapshot version has been looking good with the TCK. 

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Shawn Jiang (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61304#action_61304 ] 

Shawn Jiang commented on AMQ-2840:
----------------------------------

Gary,  #987193  bring the same TCK Failures.    Could you revert them ?

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Kevan Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61310#action_61310 ] 

Kevan Miller commented on AMQ-2840:
-----------------------------------

Shawn, thanks for running those tests!

We're also waiting for an Axis release. But I'm hoping for a mid-September release. So, a 5.4.1 in 2-3 weeks should work. 

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Eugene Rodos (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61045#action_61045 ] 

Eugene Rodos commented on AMQ-2840:
-----------------------------------

Any way this fix can be included in 5.4.0?  This seems like a glaring omission of very basic functionality and the fix looks simple enough.

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>             Fix For: 5.4.0
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Shawn Jiang (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61273#action_61273 ] 

Shawn Jiang commented on AMQ-2840:
----------------------------------

Hi, this patch broke the jms spec:

-----------------------------------------------------------------------------------------------
/**Returns an Enumeration of all the property names. 
** Note that *JMS standard header fields are not considered properties and are not returned in this enumeration*. 
**/
Enumeration getPropertyNames()  throws JMSException
----------------------------------------------------------------------------------------------------

So that this change is preventing the tck from passing.    Could you please fix this ?  


> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.0
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Gary Tully (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61293#action_61293 ] 

Gary Tully commented on AMQ-2840:
---------------------------------

Eugene, that seems correct, have made is so JMSX props are returned. r987193

Shawn, will this pass the tck?

As to a 5.4.1 asap, I agree it is a portability issue, but one few people are likely to encounter. Lets see what others think.

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Gary Tully (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gary Tully resolved AMQ-2840.
-----------------------------

    Resolution: Fixed

resolving this. It may make sense to validate tck again, just to be sure to be sure.

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Andriy Hnativ (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andriy Hnativ updated AMQ-2840:
-------------------------------

    Attachment: patch.txt

The patch

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>             Fix For: 5.4.0
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Gary Tully (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gary Tully reassigned AMQ-2840:
-------------------------------

    Assignee: Gary Tully

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.0
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Kevan Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61301#action_61301 ] 

Kevan Miller commented on AMQ-2840:
-----------------------------------

In Geronimo, we've been lining up our next 2.2.x release to be based on 5.4.0. With the limited time between this regression and release, we weren't able to nail down the cause of this problem (we had made connector/transaction manager updates around the same time, which could have had some impact on our JMS tests).

So, I, for one, would be interested in a 5.4.1 release "soon".

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Reopened: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Eugene Rodos (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Eugene Rodos reopened AMQ-2840:
-------------------------------


> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Reopened: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Gary Tully (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gary Tully reopened AMQ-2840:
-----------------------------

    Regression: [Regression]

pending spec compliance issue - part of this patch needs to be reverted.

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.0
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Gary Tully (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61305#action_61305 ] 

Gary Tully commented on AMQ-2840:
---------------------------------

I reverted the changes so the TCK will pass. I added a new method, getAllPropertyNames() that returns all the properties, JMS standard props and JMSX extension props, so folks who want this non standard behavior can have it in an explicitly non portable way.

My bad on this late change, not sure it would have been avoided if I validated against the spec though, I would have taken Eugene's interpretation.

What is the expected drop date for Geronimo, it makes sense to cut a new release for the next 2.2.x release. Think it may also makes sense to wait some time to ensure there are no other must fix issues.


> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Shawn Jiang (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61291#action_61291 ] 

Shawn Jiang commented on AMQ-2840:
----------------------------------

I found that the version of trunk is 5.5-SNAPSHOT.    Are you planing to release a 5.4.1 to fix this ?

In my opinion,  a 5.4.1 with fix should be released ASAP because the user's JMS code based on the wrong implementation will not be portable.


> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Eugene Rodos (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61292#action_61292 ] 

Eugene Rodos commented on AMQ-2840:
-----------------------------------

Guys, let's read the JMS spec carefully:

The spec says that the following are standard header fields (starting with "JMS" prefix):
JMSDestination
JMSDeliveryMode
JMSExpiration
JMSPriority
JMSMessageID
JMSTimestamp
JMSCorrelationID
JMSReplyTo
JMSType
JMSRedelivered

However, the following are NOT header fields but are JMS Defined *_Properties_* (starting with "JMSX" prefix):
JMSXUserID
JMSXAppID
JMSXDeliveryCount
JMSXGroupID
JMSXGroupSeq
JMSXProducerTXID
JMSXConsumerTXID
JMSXRcvTimestamp
JMSXState

Of these, it looks like AMQ supports 3: JMSXDeliveryCount, JMSXGroupID and JMSXGroupSeq.  Now, the ActiveMQMessage class puts both the JMS header fields and the JMSX message properties into the same JMS_PROPERTY_SETERS map, so i agree, returning the whole map is wrong, but the JMSX properties MUST still be returned.

So I think the proper solution is to replace the (reverted) line:
{code}
result.addAll(new Vector<String>(JMS_PROPERTY_SETERS.keySet()));
{code}
with something like:
{code}
for (String propName : JMS_PROPERTY_SETERS.keySet()) {
    if (propName.startsWith("JMSX")) {
        propsVector.add(propName);
    }
}
{code}


> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.1
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (AMQ-2840) JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()

Posted by "Gary Tully (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/AMQ-2840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gary Tully resolved AMQ-2840.
-----------------------------

    Resolution: Fixed

patch applied in r983564 with thanks. added a test to keep the changes safe!

> JMSX Properties not returned from message.propertyExists() and message.getPropertyNames()
> -----------------------------------------------------------------------------------------
>
>                 Key: AMQ-2840
>                 URL: https://issues.apache.org/activemq/browse/AMQ-2840
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.4.0
>            Reporter: Andriy Hnativ
>            Assignee: Gary Tully
>             Fix For: 5.4.0
>
>         Attachments: patch.txt
>
>
> Methods propertyExists() and getPropertyNames() defined in the class org.apache.activemq.command.ActiveMQMessage (activemq-core) return only custom properties. The JMSX properties (like JMSXDeliveryCount) are not being returned. This problem exists not only for a new version of ActiveMQ (5.4), but also for 5.3 (and probably for others as well).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.