2014-05-12 16:39 GMT+02:00 Mark Thomas <ma...@apache.org>: > I just got to that. I've answered your immediate question but I want to > do a little more digging of my own. I'll respond in more detail on that > thread. That said, I'm not expecting NIO2 to be declared stable in this > release so I don't see this as a release blocker. >
Ok. I looked at it more, and I think the comment you made is correct: on shutdown it would simply try to read data (but the read for the keepalive is pending, so it gets an exception). So I'm not reverting anything, the code as is won't do anything that would cause real issues even if it's not perfect. The main idea was to avoid ignoring read pending exceptions. Rémy
