svn commit: r612845 - /incubator/sling/trunk/sling/scheduler/src/main/java/org/apache/sling/scheduler/impl/QuartzJobExecutor.java

[cziegeler]

Author: cziegeler
Date: Thu Jan 17 07:34:58 2008
New Revision: 612845

URL: http://svn.apache.org/viewvc?rev=612845&view=rev
Log:
Improve logging for job execution.

Modified:
    
incubator/sling/trunk/sling/scheduler/src/main/java/org/apache/sling/scheduler/impl/QuartzJobExecutor.java

Modified: 
incubator/sling/trunk/sling/scheduler/src/main/java/org/apache/sling/scheduler/impl/QuartzJobExecutor.java
URL: 
http://svn.apache.org/viewvc/incubator/sling/trunk/sling/scheduler/src/main/java/org/apache/sling/scheduler/impl/QuartzJobExecutor.java?rev=612845&r1=612844&r2=612845&view=diff
==============================================================================
--- 
incubator/sling/trunk/sling/scheduler/src/main/java/org/apache/sling/scheduler/impl/QuartzJobExecutor.java
 (original)
+++ 
incubator/sling/trunk/sling/scheduler/src/main/java/org/apache/sling/scheduler/impl/QuartzJobExecutor.java
 Thu Jan 17 07:34:58 2008
@@ -75,12 +75,16 @@
                 }
             } else if (job instanceof Runnable) {
                 ((Runnable) job).run();
+            } else {
+                logger.error("Scheduled job {} is neither a job nor a 
runnable.", job);
             }
         } catch (final Throwable t) {
-
+            // if this is a quartz exception, rethrow it
             if (t instanceof JobExecutionException) {
                 throw (JobExecutionException) t;
             }
+            // there is nothing we can do here, so we just log
+            logger.error("Exception during job execution of " + job + " : " + 
t.getMessage(), t);
         } finally {
 
             this.release(data);