symat commented on a change in pull request #1301: ZOOKEEPER-3774: Close quorum socket asynchronously on the leader to a… URL: https://github.com/apache/zookeeper/pull/1301#discussion_r410023339
########## File path: zookeeper-docs/src/main/resources/markdown/zookeeperAdmin.md ########## @@ -1497,7 +1501,8 @@ and [SASL authentication for ZooKeeper](https://cwiki.apache.org/confluence/disp * *sslQuorum* : (Java system property: **zookeeper.sslQuorum**) **New in 3.5.5:** - Enables encrypted quorum communication. Default is `false`. + Enables encrypted quorum communication. Default is `false`. When enabling this feature, please also consider enabling *leader.closeSocketAsync* + and *learner.closeSocketAsync* to avoid issues associated with the potentially long socket closing time when shutting down an SSL connection. Review comment: I haven't found `learner.closeSocketAsync` in the code. Did I miss something, or should we not mention that property here? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
