Thanks again for the review @zentol. I had to add the `IdleStateHandler` because it happened sometimes in `RestClientTest#testConnectionClosedHandling` that the channel would not be closed by Netty after we closed the server socket. So sometimes Netty detects such an event and then calls `channelInactive` and sometimes not. For the latter case, we need to monitor the connection for idleness.
[ Full content available at: https://github.com/apache/flink/pull/6763 ] This message was relayed via gitbox.apache.org for [email protected]
