buptubuntu commented on pull request #2497: URL: https://github.com/apache/thrift/pull/2497#issuecomment-1019429692
> @buptubuntu could you please clarify whether you are working on the new approach(-es)? As I mentioned above: Yeah，I think there are generally two solutions: * add a func called: CloseImmediatly，it will close all clients, that will cause client err， but it is a seperate func, and user can choose. * add a var called ServerCloseTimeout like the ServerConnectivityCheckInterval，and its default value is 0，means that the server will wait all client closed even the underlying net.Conn is hanged; or if user set ServerCloseTimeout positive, the server will wait for a while before close all the clients @fishy @dcelasun -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
