[
https://issues.apache.org/jira/browse/QPID-6701?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14706717#comment-14706717
]
Rajith Attapattu commented on QPID-6701:
----------------------------------------
Couple of points,
1. Assert was to verify if the node has the supplied properties.Ex durable or
if it's a queue or exchange.
2. Historically if a node didn't resolved, an exception was thrown. Whether
this was right or wrong, this is behavior that users had relied upon. Changing
that is cuasing issues.with existing customers, which is why the users are
treating it as a regression.
3. While a consume might fail, a producer will not. It will continue to send
and the exchange will drop the messages.
4. If you want to allow addresses that are neither queues nor topics, then make
that behavior explicitly configurable and leave the existing behavior the
default.
> [Regression btw 0.30 - 0.32] If address doesn't resolve an exception is not
> thrown
> ----------------------------------------------------------------------------------
>
> Key: QPID-6701
> URL: https://issues.apache.org/jira/browse/QPID-6701
> Project: Qpid
> Issue Type: Bug
> Components: Java Client
> Affects Versions: 0.32
> Reporter: Rajith Attapattu
> Assignee: Keith Wall
> Priority: Blocker
> Fix For: qpid-java-6.0
>
>
> If you run "java -cp $CP org.apache.qpid.example.Spout non-existing-node",
> 1. In 0.30 you get an exception with the cause "org.apache.qpid.AMQException:
> Exception occured while verifying destination"
> 2. In 0.32 no such exception is thrown.
> The issue is in the resolveAddress method in AMQSession class.
> If resolved is false no action is taken. There are a couple issues with this.
> 1. A producer can be created to a non existent queue or exchange.
> 2. Messages being dropped - While sending to a non existing exchange will
> result in an error, sending to a non existent queue via an exchange will
> simply result in messages being dropped.
> 3. The address will continue to be resolved as there was no error the
> previous time.
> We should throw an exception if resolved == false.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
