qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (QPID-6612) [Java Broker] Refactor Connection(Adapter), ProtocolEngine and AMQConnectionModel
Date Mon, 27 Jul 2015 09:06:04 GMT

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

ASF subversion and git services commented on QPID-6612:
-------------------------------------------------------

Commit 1692839 from [~godfrer] in branch 'java/trunk'
[ https://svn.apache.org/r1692839 ]

QPID-6612 : Remove duplicated addition of connection principal (also added in the base class

> [Java Broker] Refactor Connection(Adapter), ProtocolEngine and AMQConnectionModel
> ---------------------------------------------------------------------------------
>
>                 Key: QPID-6612
>                 URL: https://issues.apache.org/jira/browse/QPID-6612
>             Project: Qpid
>          Issue Type: Improvement
>          Components: Java Broker
>            Reporter: Rob Godfrey
>            Assignee: Rob Godfrey
>             Fix For: 6.0 [Java]
>
>
> The modelling of (AMQP) connections within the broker is a bit of a mess with code duplicated
between classes because of a lack of a common base class for the connection objects.
> Refactor these classes to have the ProtocolEngine evolve into the (Model object) AMQPConnection.
 Remove the need for AMQConnectionModel and ConnectionAdapter.  Protocol specific classes
like ServerConnection should be invisible outside the protocol and be encapsulated / delegated
to by the AMQPConnection variant



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

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


Mime
View raw message