Author: fhanik Date: Fri Aug 10 14:07:46 2007 New Revision: 564768 URL: http://svn.apache.org/viewvc?view=rev&rev=564768 Log: Fix the comet recycle sync issue, will think of another solution some other time, for now this solves the problem
Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java?view=diff&rev=564768&r1=564767&r2=564768 ============================================================================== --- tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java (original) +++ tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java Fri Aug 10 14:07:46 2007 @@ -989,6 +989,16 @@ localName = null; remotePort = -1; localPort = -1; + //fix the synchronization scenario due to + //dual comet flags. + //while the response/request + //might already be recycled, this circumvents the bug + //and should not be an expensive operation + //however, this is a TODO and FIXME + //as it would be better coordinate the recycling of the request/response + //instead + response.recycle(); + request.recycle(); } --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]