Remove close(..) call which gave us troubles with

ClosedChannelException. See #142 and #138
netty · Jan 11, 2012 · 02be252 · 02be252
1 parent e2109b2
commit 02be252
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/main/java/org/jboss/netty/channel/socket/nio/NioClientSocketPipelineSink.java b/src/main/java/org/jboss/netty/channel/socket/nio/NioClientSocketPipelineSink.java
@@ -370,7 +370,14 @@ private void processConnectTimeout(Set<SelectionKey> keys, long currentTimeNanos
             ConnectException cause = null;
             for (SelectionKey k: keys) {
                 if (!k.isValid()) {
-                    close(k);
+                    // Comment the close call again as it gave us major problems
+                    // with ClosedChannelExceptions.
+                    //
+                    // See:
+                    // * https://github.com/netty/netty/issues/142
+                    // * https://github.com/netty/netty/issues/138
+                    //
+                    // close(k);
                     continue;
                 }