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

Robbie Gemmell commented on QPID-4521:
--------------------------------------

Change committed in http://svn.apache.org/viewvc?rev=1425515&view=rev which 
corrects the defect and adds unit tests for the InboundMessageAdapter to verify 
it is returning the correct values.

I didn't add any system test using an alternate topic exchange (although I did 
actually write such a test to initially confirm the defect) as both the topic 
exchange itself and the alternate exchange functionality are already system 
tested quite heavily (mostly in the python tests but a couple in Java, its just 
that they use Fanout exchanges for the alternate exchanges, which is a far more 
typical use case) and this is simply a trivial defect in a piece of code that 
works between the two.
                
> [Java broker] routing to an alternate exchange during queue deletion fails 
> for topic exchanges.
> -----------------------------------------------------------------------------------------------
>
>                 Key: QPID-4521
>                 URL: https://issues.apache.org/jira/browse/QPID-4521
>             Project: Qpid
>          Issue Type: Bug
>          Components: Java Broker
>    Affects Versions: 0.16, 0.18, 0.20
>            Reporter: Robbie Gemmell
>            Assignee: Robbie Gemmell
>            Priority: Minor
>             Fix For: 0.21
>
>
> Routing to an alternate exchange during queue deletion fails for topic 
> exchanges. This is because the InboundMessageAdapter used to adapt the queues 
> QueueEntry for routing by the exchange has a defect in the 
> getRountingKeyShortSring() method. This method returns an AMQShortString and 
> is used by the Topic exchange (to avoid redoing much of the Topic matching 
> logic that uses AMQShortString) whereas the other exchanges use the 
> getRoutingKey() method which returns a String.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to