You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2013/05/16 09:36:17 UTC

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

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);
                 }
             }
         }