[GEODE-3079] Ensure new protocol shuts down cleanly

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Component/s: client/server
• Assignee: Alexander Murmann
• Reporter: Galen O'Sullivan
• Created: 14/Jun/17 23:15
• Updated: 20/Apr/18 19:52
• Resolved: 14/Sep/17 23:23

Description

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.

Issue Links

• is depended upon by: GEODE-3170 Geode keeps logging org.apache.geode.protocol.exception.InvalidProtocolMessageException
• relates to: GEODE-3086 NewClientServerConnection and loadEmergencyClasses()