Re: svn commit: r835668 - /incubator/olio/webapp/java/trunk/ws/apps/webapp/src/java/org/apache/olio/webapp/util/ContentCacheResponseWrapper.java

[Sheetal Patil]

No, I have not tested the fix for glassfish.
But, I think it should not cause any problem on glassfish.

Thanks and Regards,
Sheetal

On 11/13/09 08:58, Shanti Subramanyam wrote:

Sheetal,
Did you check this fix against glassfish ?

Shanti

shee...@apache.org wrote:

Author: sheetal
Date: Fri Nov 13 00:32:57 2009
New Revision: 835668

URL: http://svn.apache.org/viewvc?rev=835668&view=rev
Log:
OLIO-135 ServletResponse flushBuffer() caused the response to commit.

Modified:

incubator/olio/webapp/java/trunk/ws/apps/webapp/src/java/org/apache/olio/webapp/util/ContentCacheResponseWrapper.java

Modified: incubator/olio/webapp/java/trunk/ws/apps/webapp/src/java/org/apache/olio/webapp/util/ContentCacheResponseWrapper.java URL: http://svn.apache.org/viewvc/incubator/olio/webapp/java/trunk/ws/apps/webapp/src/java/org/apache/olio/webapp/util/ContentCacheResponseWrapper.java?rev=835668&r1=835667&r2=835668&view=diff ============================================================================== --- incubator/olio/webapp/java/trunk/ws/apps/webapp/src/java/org/apache/olio/webapp/util/ContentCacheResponseWrapper.java (original) +++ incubator/olio/webapp/java/trunk/ws/apps/webapp/src/java/org/apache/olio/webapp/util/ContentCacheResponseWrapper.java Fri Nov 13 00:32:57 2009

@@ -88,11 +88,11 @@

@Override public void setBufferSize(int i) {

-        System.err.println("setBufferSize(int i) called.");
+        //System.err.println("setBufferSize(int i) called.");
     }

@Override public void flushBuffer() throws IOException {

-        System.err.println("flushBuffer() called.");
+        //System.err.println("flushBuffer() called.");
         stream.flush();
     }