activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Craig Lewis (JIRA)" <j...@apache.org>
Subject [jira] Commented: (AMQ-3172) Add support for Message Priority to Stomp
Date Tue, 08 Feb 2011 02:12:57 GMT

    [ https://issues.apache.org/jira/browse/AMQ-3172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12991770#comment-12991770
] 

Craig Lewis commented on AMQ-3172:
----------------------------------

Confirmed.  Without the sleep, it fails approximately 50% of the time.  After adding the sleep,
it's passed 10 runs out of 10.

Thanks for the help.

> Add support for Message Priority to Stomp
> -----------------------------------------
>
>                 Key: AMQ-3172
>                 URL: https://issues.apache.org/jira/browse/AMQ-3172
>             Project: ActiveMQ
>          Issue Type: Improvement
>          Components: Transport
>    Affects Versions: 5.4.2
>            Reporter: Craig Lewis
>            Priority: Minor
>         Attachments: StompTest.java.patch
>
>
> Creating a Stomp message with a priority header has no effect.  The messages are delivered
in FIFO order, not priority order.
> The Stomp transport (activemq-core/src/main/java/org/apache/activemq/transport/stomp/FrameTranslator.java)
is calling msg.setJMSPriority(), but never calls producer.setPriority().  Because of this,
messages claims to have a priority, but the messages are never actually re-ordered.
> I believe a new unit test should be added to activemq-core/src/test/java/org/apache/activemq/transport/stomp/StompTest.java.
 It should send 2 messages at different priorities, and verifies that they are re-ordered
by ActiveMQ.  I'll attach a unit test.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message