svn commit: r1483220 - in /sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed: ScheduleInfo.java TimedEventReceiver.java

[cziegeler]

Author: cziegeler
Date: Thu May 16 07:36:17 2013
New Revision: 1483220

URL: http://svn.apache.org/r1483220
Log:
SLING-2868 :  Improve and cleanup implementation : Improve logging for timed 
events

Modified:
    
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/ScheduleInfo.java
    
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventReceiver.java

Modified: 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/ScheduleInfo.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/ScheduleInfo.java?rev=1483220&r1=1483219&r2=1483220&view=diff
==============================================================================
--- 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/ScheduleInfo.java
 (original)
+++ 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/ScheduleInfo.java
 Thu May 16 07:36:17 2013
@@ -61,7 +61,7 @@ final class ScheduleInfo implements Seri
         // if the event contains a timed event id or a job id we'll append 
that to the name
         this.topic = 
(String)event.getProperty(EventUtil.PROPERTY_TIMED_EVENT_TOPIC);
         if ( topic == null ) {
-            throw new IllegalArgumentException("Timed event does not contain 
required property " + EventUtil.PROPERTY_TIMED_EVENT_TOPIC);
+            throw new IllegalArgumentException("Timed event does not contain 
required property " + EventUtil.PROPERTY_TIMED_EVENT_TOPIC + " : " +  
EventUtil.toString(event));
         }
         // TODO id or job name needs to be available?!?!
         final String id = 
(String)event.getProperty(EventUtil.PROPERTY_TIMED_EVENT_ID);

Modified: 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventReceiver.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventReceiver.java?rev=1483220&r1=1483219&r2=1483220&view=diff
==============================================================================
--- 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventReceiver.java
 (original)
+++ 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventReceiver.java
 Thu May 16 07:36:17 2013
@@ -150,7 +150,7 @@ public class TimedEventReceiver implemen
                     scheduleInfo = new ScheduleInfo(event);
                     this.writeEvent(event, scheduleInfo);
                 } catch (final IllegalArgumentException iae) {
-                    this.logger.error(iae.getMessage());
+                    this.logger.error(iae.getMessage(), iae);
                 }
             }
         }