[
https://issues.apache.org/jira/browse/GEODE-3075?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16058246#comment-16058246
]
ASF GitHub Bot commented on GEODE-3075:
---------------------------------------
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/586
@bschuchardt Was that the only failure? It looks like another test had set
`mcast-port` to a non-zero value and failed to clean up the environment
I'm rerunning precheckin; will continue working on top of this branch and
hopefully this passes. If that's the only failure, I would think it should be
safe to merge with the one-line added change.
> Bifurcate ServerConnection, create factory and feature toggle
> -------------------------------------------------------------
>
> Key: GEODE-3075
> URL: https://issues.apache.org/jira/browse/GEODE-3075
> Project: Geode
> Issue Type: Sub-task
> Components: client/server
> Reporter: Galen O'Sullivan
>
> Make a factory that creates instances of the new subclasses of
> {{ServerConnection}} : {{NewClientServerConnection}} and
> {{LegacyServerConnection}}. This will be the basis for future refactors. This
> should involve minimal changes to {{AcceptorImpl}} and a minimal
> implementation of {{NewClientServerConnection}}. All the existing
> functionality of {{ServerConnection}} must still work.
> To do this, we will need a feature toggle so that the new client protocol
> (which will now be in a broken state) never gets run in production, but can
> be enabled for testing. This should probably be set via a hidden system
> property and not through any config that is a part of the public API.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
