You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by to...@apache.org on 2016/05/24 12:53:13 UTC

svn commit: r1745354 - /sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/queue/impl/simple/SimpleDistributionQueueCheckpoint.java

Author: tommaso
Date: Tue May 24 12:53:13 2016
New Revision: 1745354

URL: http://svn.apache.org/viewvc?rev=1745354&view=rev
Log:
SLING-5737 - log an error if checkpointing fails, no runtime exception

Modified:
    sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/queue/impl/simple/SimpleDistributionQueueCheckpoint.java

Modified: sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/queue/impl/simple/SimpleDistributionQueueCheckpoint.java
URL: http://svn.apache.org/viewvc/sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/queue/impl/simple/SimpleDistributionQueueCheckpoint.java?rev=1745354&r1=1745353&r2=1745354&view=diff
==============================================================================
--- sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/queue/impl/simple/SimpleDistributionQueueCheckpoint.java (original)
+++ sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/queue/impl/simple/SimpleDistributionQueueCheckpoint.java Tue May 24 12:53:13 2016
@@ -82,7 +82,7 @@ class SimpleDistributionQueueCheckpoint
             boolean success = checkpointFile.renameTo(new File(checkpointDirectory, fileName));
             log.debug("checkpoint succeeded: {}", success);
         } catch (Exception e) {
-            throw new RuntimeException(e);
+            log.error("failed checkpointing for queue {}", queue.getName());
         }
     }
 }