DonalEvans commented on pull request #6682: URL: https://github.com/apache/geode/pull/6682#issuecomment-877306296
> Since `IOException`s are checked, the implication is that they may be recoverable. However, in this case since netty is telling us about this problem, there isn't anything we can do about it. We're being told that the client is gone and we can perform any necessary cleanup. Aah, I see now. As long as we can be sure that we only see an `IOException` when the client is gone, and not in other cases, then it should be fine to just revert the entire commit for GEODE-9411. If we think there might be cases where an `IOException` is thrown and we don't want to close the connection, maybe we could check for the "Connection reset by peer" message in the exception and close if we see that? -- 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]
