[ 
https://issues.apache.org/jira/browse/GEODE-6573?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Darrel Schneider reassigned GEODE-6573:
---------------------------------------

    Assignee: Darrel Schneider

> ConnectionManager has some methods on it that should be on Connection instead
> -----------------------------------------------------------------------------
>
>                 Key: GEODE-6573
>                 URL: https://issues.apache.org/jira/browse/GEODE-6573
>             Project: Geode
>          Issue Type: Improvement
>          Components: client/server
>            Reporter: Darrel Schneider
>            Assignee: Darrel Schneider
>            Priority: Major
>
> org.apache.geode.cache.client.internal.pooling.ConnectionManager has three 
> methods on it that do not belong: getConnection(Connection), 
> activate(Connection), passivate(Connection, boolean).
> All three of these methods should instead be on the Connection interface. 
> None of them have anything to do with the manager.
> getConnection(Connection) is requesting the wrapped connection from a 
> connection. So a better name for it would be getWrappedConnection.
> Not all implementations of the Connection interface need to support wrapping 
> or activation but those implementations can just implement these operations 
> as a noop.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to