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]

Reply via email to