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