You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commons-dev@ws.apache.org by ve...@apache.org on 2009/06/15 00:03:52 UTC

svn commit: r784627 - /webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java

Author: veithen
Date: Sun Jun 14 22:03:52 2009
New Revision: 784627

URL: http://svn.apache.org/viewvc?rev=784627&view=rev
Log:
Added review comment.

Modified:
    webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java

Modified: webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java?rev=784627&r1=784626&r2=784627&view=diff
==============================================================================
--- webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java (original)
+++ webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java Sun Jun 14 22:03:52 2009
@@ -105,6 +105,7 @@
         }
 
         // has this message already expired? expiration time == 0 means never expires
+        // TODO: explain why this is necessary; normally it is the responsibility of the provider to handle message expiration
         try {
             long expiryTime = message.getJMSExpiration();
             if (expiryTime > 0 && System.currentTimeMillis() > expiryTime) {