[jira] [Commented] (GEODE-3079) Ensure new protocol shuts down cleanly

["ASF subversion and git services (JIRA)" <ji...@apache.org>]

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

ASF subversion and git services commented on GEODE-3079:
--------------------------------------------------------

Commit 6bbaad05274117bfe20f2991b09150442bafa50a in geode's branch refs/heads/develop from [~amurmann]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=6bbaad0 ]

GEODE-3079 Ensure emergencyClose() closes socket


> Ensure new protocol shuts down cleanly
> --------------------------------------
>
>                 Key: GEODE-3079
>                 URL: https://issues.apache.org/jira/browse/GEODE-3079
>             Project: Geode
>          Issue Type: Sub-task
>          Components: client/server
>            Reporter: Galen O'Sullivan
>            Assignee: Galen O'Sullivan
>
> As a user of the new protocol, I need to be sure my servers shut down cleanly in order to manage my grid.  Ensure new protocol shuts down cleanly and does not impact server restarts/shutdowns.
> Specifically, investigate:
> Implement at least the {{terminateNow()}}, {{emergencyClose()}}, and {{cleanup()}} methods (and do a check for others) on {{NewClientServerConnection}}. Make sure that we handle each one in a similar fashion to how the legacy {{ServerConnection}} does.
> AcceptorImpl and its callers use loadEmergencyClasses() to make sure that classes which are needed for a proper shutdown are loaded when the Cache starts, and prior to shutdown (which could be an OOM or similar situation).
> In the parent call (I think AcceptorImpl) add a call to our own method, and call the same method on any classes we depend on.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)