[ZOOKEEPER-3682] Stop initializing new SSL connection if ZK server is shutting down

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.7.0
• Component/s: server
• Labels: pull-request-available
• Assignee: Unassigned Assign to me
• Reporter: Jie Huang
• Created: 08/Jan/20 16:34
• Updated: 26/Jan/22 08:47
• Resolved: 26/Feb/20 21:42

Description

ZK keeps accepting new connections while it's being shut down then immediately close them when it finds out that the ZK server is not running. It's not a big deal before SSL is enabled since creating TCP connections is relatively cheap.
 
With SSL being widely enabled,  creating SSL connections involves handshake that takes non-trivial CPU time, which is wasted since the connections are closed right after. 
 
This JIRA is going to stop initializing TLS handshake if the zkServer is not serving to save resources.

Issue Links

• is related to: ZOOKEEPER-3988 org.apache.zookeeper.server.NettyServerCnxn.receiveMessage throws NullPointerException
• is related to: ZOOKEEPER-4453 NettyServerCnxnFactory: allow to configure the early TLS connection drop feature
• links to: GitHub Pull Request #1210