You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by sk...@apache.org on 2009/02/22 12:20:44 UTC

svn commit: r746656 - /myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/servlet/ConversationManagerSessionListener.java

Author: skitching
Date: Sun Feb 22 11:20:43 2009
New Revision: 746656

URL: http://svn.apache.org/viewvc?rev=746656&view=rev
Log:
trivial changes: 
* Set thread name for ConversationWiperThread for debugging convenience
* add comment

Modified:
    myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/servlet/ConversationManagerSessionListener.java

Modified: myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/servlet/ConversationManagerSessionListener.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/servlet/ConversationManagerSessionListener.java?rev=746656&r1=746655&r2=746656&view=diff
==============================================================================
--- myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/servlet/ConversationManagerSessionListener.java (original)
+++ myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/servlet/ConversationManagerSessionListener.java Sun Feb 22 11:20:43 2009
@@ -60,6 +60,9 @@
  * See {@link org.apache.myfaces.orchestra.conversation.ConversationWiperThread}
  * for more details.
  */
+// TODO: rename this class to ConversationWiperThreadManager or similar; it is not just a
+// SessionListener as it also implements ServletContextListener. This class specifically
+// handles ConversationWiperThread issues...
 public class ConversationManagerSessionListener
     implements HttpSessionAttributeListener, HttpSessionActivationListener, ServletContextListener
 {
@@ -83,6 +86,7 @@
         if (conversationWiperThread == null)
         {
             conversationWiperThread = new ConversationWiperThread(checkTime);
+            conversationWiperThread.setName("Orchestra:ConversationWiperThread");
             conversationWiperThread.start();
         }
         else