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

Robbie Gemmell edited comment on QPID-7605 at 1/4/17 5:04 PM:
--------------------------------------------------------------

{quote}
So the info map is on the close... my comments were focussed on the open...
{quote}
Yep, just checking you din't mean using a special container-id in the response 
would be conveying the same detail we added the close error info map for. 
Rather it would only be signalling the 'close is about to arrive' behaviour, 
and the fact both  bits reference the container-id here is unrelated.

{quote}
Given that we do not originally call out that the empty string should be 
treated as a special "I'm sorry Dave, I'm afraid I can't do that" meaning I'm 
hesitant about saying that that in itself should be enough to call out the fact 
that the connection is going to immediately close... What we could say though 
is that we define a connection capability "EMPTY_CONTAINER_ID_ON_OPEN_FAILS" or 
something such that if the capability is present and the container-id is not 
empty the the initiator can be assured that the connection is not about to 
immediately close... and if the capability is there and the container-id is the 
empty string then it knows that the connection is about to close.
{quote}

Yes, thats why we went with the property originally to convey the impending 
close, as the container-id behaviour had no definition around it like that in 
the original spec.We also thought it could be used to the same end in any 
similar behaviour-specific mechanisms to the ClientID scenario. Making it an 
even more general mechanism like this isn't something I'd considered before, 
but that also makes sense. The only issue would be if anyone wanted the actual 
container-id of the remote peer to help govern their subsequent behaviour in 
some way, not sure how likely that is.



was (Author: gemmellr):
{quote}
So the info map is on the close... my comments were focussed on the open...
{quote}
Yep, just checking you din't mean using a special container-id in the response 
would be conveying the same detail we added the close error info map for. 
Rather it would only be signalling the 'close is about to arrive' behaviour, 
and the fact both  bits reference the container-id here is unrelated.

{quote}
Given that we do not originally call out that the empty string should be 
treated as a special "I'm sorry Dave, I'm afraid I can't do that" meaning I'm 
hesitant about saying that that in itself should be enough to call out the fact 
that the connection is going to immediately close... What we could say though 
is that we define a connection capability "EMPTY_CONTAINER_ID_ON_OPEN_FAILS" or 
something such that if the capability is present and the container-id is not 
null the the initiator can be assured that the connection is not about to 
immediately close... and if the capability is there and the container-id is the 
empty string then it knows that the connection is about to close.
{quote}

Yes, thats why we went with the property originally to convey the impending 
close, as the container-id behaviour had no definition around it like that in 
the original spec.We also thought it could be used to the same end in any 
similar behaviour-specific mechanisms to the ClientID scenario. Making it an 
even more general mechanism like this isn't something I'd considered before, 
but that also makes sense.

I see you've edited this already <removes question> :)

> [Java Broker] [AMQP1.0] Container id uniqueness
> -----------------------------------------------
>
>                 Key: QPID-7605
>                 URL: https://issues.apache.org/jira/browse/QPID-7605
>             Project: Qpid
>          Issue Type: Improvement
>          Components: Java Broker
>            Reporter: Keith Wall
>             Fix For: qpid-java-7.0
>
>
> The AMQP 1.0 protocol layer implementation must ensure that the AMQP Open 
> performative container-id is unique amongst existing established connections.
> As the JMS client id maps to the container-id, so this will fulfil the JMS 
> requirement.  
> https://docs.oracle.com/javaee/7/api/javax/jms/Connection.html#setClientID-java.lang.String-
> Note that the Qpid JMS Client requires the Close performative with an Error 
> containing a hint to generate to correct JMS exception.  How will the Qpid 
> Broker know to do this?
> org.apache.qpid.jms.integration.FailedConnectionsIntegrationTest#testConnectWithInvalidClientIdThrowsICIDEWhenInvalidContainerHintPresent



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org

Reply via email to