arturobernalg commented on PR #1088: URL: https://github.com/apache/tomcat/pull/1088#issuecomment-6860588994
-1 This introduces a regression when switching from UDP to TCP for the same cluster member. `ParallelNioSender` reuses the same `NioSender`, but after a UDP send the channel remains connected. A subsequent TCP send fails with a `NullPointerException` because `socketChannel` is null. I reproduced this with running Tomcat instances. All three UDP-to-TCP transitions succeeded on main, but failed with this change. There also appears to be a resource leak: the UDP socket remains open after `tomcat.stop()`. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
