You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2014/04/20 16:43:52 UTC

svn commit: r1588771 - in /tomcat/trunk/java/org/apache/coyote/http11: InternalNio2OutputBuffer.java upgrade/Nio2ServletOutputStream.java

Author: remm
Date: Sun Apr 20 14:43:52 2014
New Revision: 1588771

URL: http://svn.apache.org/r1588771
Log:
When not inline, do not use a new thread since the completion handler is already run in a new thread.

Modified:
    tomcat/trunk/java/org/apache/coyote/http11/InternalNio2OutputBuffer.java
    tomcat/trunk/java/org/apache/coyote/http11/upgrade/Nio2ServletOutputStream.java

Modified: tomcat/trunk/java/org/apache/coyote/http11/InternalNio2OutputBuffer.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/InternalNio2OutputBuffer.java?rev=1588771&r1=1588770&r2=1588771&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http11/InternalNio2OutputBuffer.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http11/InternalNio2OutputBuffer.java Sun Apr 20 14:43:52 2014
@@ -145,7 +145,7 @@ public class InternalNio2OutputBuffer ex
                     }
                 }
                 if (notify) {
-                    endpoint.processSocket(socket, SocketStatus.OPEN_WRITE, true);
+                    endpoint.processSocket(socket, SocketStatus.OPEN_WRITE, false);
                 }
             }
 
@@ -196,7 +196,7 @@ public class InternalNio2OutputBuffer ex
                     }
                 }
                 if (notify) {
-                    endpoint.processSocket(socket, SocketStatus.OPEN_WRITE, true);
+                    endpoint.processSocket(socket, SocketStatus.OPEN_WRITE, false);
                 }
             }
 

Modified: tomcat/trunk/java/org/apache/coyote/http11/upgrade/Nio2ServletOutputStream.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/Nio2ServletOutputStream.java?rev=1588771&r1=1588770&r2=1588771&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http11/upgrade/Nio2ServletOutputStream.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/Nio2ServletOutputStream.java Sun Apr 20 14:43:52 2014
@@ -62,7 +62,7 @@ public class Nio2ServletOutputStream ext
                         } catch (IOException e) {
                             socketWrapper.setError(true);
                             onError(e);
-                            endpoint.processSocket(socketWrapper, SocketStatus.ERROR, true);
+                            endpoint.processSocket(socketWrapper, SocketStatus.ERROR, false);
                         }
                     }
                 }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org