On 12/05/2014 21:00, Rémy Maucherat wrote: > 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.
I'm running the unit tests now. I want to look at BZ 56516 some more and I don't want to hold up a 8.0.x release any longer. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
