Author: cziegeler
Date: Tue Jul 23 08:02:36 2013
New Revision: 1505926

URL: http://svn.apache.org/r1505926
Log:
Add simpler addJob() method

Modified:
    
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java
    
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/jobs/JobManager.java

Modified: 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java?rev=1505926&r1=1505925&r2=1505926&view=diff
==============================================================================
--- 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java
 (original)
+++ 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java
 Tue Jul 23 08:02:36 2013
@@ -775,6 +775,14 @@ public class JobManagerImpl
     }
 
     /**
+     * @see org.apache.sling.event.jobs.JobManager#addJob(java.lang.String, 
java.util.Map)
+     */
+    @Override
+    public Job addJob(String topic, Map<String, Object> properties) {
+        return this.addJob(topic, null, properties);
+    }
+
+    /**
      * @see org.apache.sling.event.jobs.JobManager#addJob(java.lang.String, 
java.lang.String, java.util.Map)
      */
     @Override

Modified: 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/jobs/JobManager.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/jobs/JobManager.java?rev=1505926&r1=1505925&r2=1505926&view=diff
==============================================================================
--- 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/jobs/JobManager.java
 (original)
+++ 
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/jobs/JobManager.java
 Tue Jul 23 08:02:36 2013
@@ -150,6 +150,23 @@ public interface JobManager {
      * If the topic is missing or illegal, no job is created and 
<code>null</code> is returned.
      *
      * @param topic The required job topic.
+     * @param properties Optional job properties
+     * @return The new job - or <code>null</code> if the job could not be 
created.
+     * @since 1.2
+     */
+    Job addJob(String topic, Map<String, Object> properties);
+
+    /**
+     * Add a new job
+     *
+     * If the topic is missing or illegal, no job is created and 
<code>null</code> is returned.
+     * This method allows to specify a job name which should uniquely identify 
this job. If a job with
+     * the same name is started on different instances, the job is still 
processed only once. However,
+     * the topology api in combination with the leader selection provides a 
better way for
+     * dealing with this situation and as jobs with name come with a heavy 
processing overhead
+     * these should be avoided.
+     *
+     * @param topic The required job topic.
      * @param name  Optional unique job name
      * @param properties Optional job properties
      * @return The new job - or <code>null</code> if the job could not be 
created.


Reply via email to