You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by fm...@apache.org on 2010/08/26 22:10:03 UTC

svn commit: r989912 - /sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingMainServlet.java

Author: fmeschbe
Date: Thu Aug 26 20:10:03 2010
New Revision: 989912

URL: http://svn.apache.org/viewvc?rev=989912&view=rev
Log:
SLING-1270 modify comment to indicate that for now we intend to keep closing the resolver here

Modified:
    sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingMainServlet.java

Modified: sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingMainServlet.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingMainServlet.java?rev=989912&r1=989911&r2=989912&view=diff
==============================================================================
--- sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingMainServlet.java (original)
+++ sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingMainServlet.java Thu Aug 26 20:10:03 2010
@@ -373,12 +373,11 @@ public class SlingMainServlet extends Ge
             // dispose any request data
             requestData.dispose();
 
-            // FIXME: This must be removed
-            // SLING-1270 Temporary solution to ensure session is logged out
+            // close the resource resolver (not relying on servlet request
+            // listener to do this for now; see SLING-1270)
             if (resolver != null) {
                 resolver.close();
             }
-            // END SLING-1270 Temporary solution to ensure session is logged out
         }
     }