svn commit: r712705 - in /synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task: TaskDescription.java TaskDescriptionFactory.java

[indika]

Author: indika
Date: Mon Nov 10 09:37:39 2008
New Revision: 712705

URL: http://svn.apache.org/viewvc?rev=712705&view=rev
Log:
Use of Genarics 

Modified:
    
synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task/TaskDescription.java
    
synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task/TaskDescriptionFactory.java

Modified: 
synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task/TaskDescription.java
URL: 
http://svn.apache.org/viewvc/synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task/TaskDescription.java?rev=712705&r1=712704&r2=712705&view=diff
==============================================================================
--- 
synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task/TaskDescription.java
 (original)
+++ 
synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task/TaskDescription.java
 Mon Nov 10 09:37:39 2008
@@ -22,10 +22,7 @@
 import org.apache.axiom.om.OMElement;
 import org.quartz.SimpleTrigger;
 
-import java.util.Date;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Set;
+import java.util.*;
 
 /**
  * Encapsulates details about a task
@@ -40,8 +37,8 @@
     private int repeatCount = SimpleTrigger.REPEAT_INDEFINITELY;
     private long repeatInterval; // in milliseconds
     private String className;
-    private List pinnedServers;
-    private Set<OMElement> xmlProperties = new HashSet<OMElement>();
+    private final List<String> pinnedServers = new ArrayList<String>();
+    private final Set<OMElement> xmlProperties = new HashSet<OMElement>();
     private String name;
     private String group;
     private Date startTime;
@@ -79,7 +76,7 @@
         xmlProperties.add(prop);
     }
 
-    public Set getProperties() {
+    public Set<OMElement> getProperties() {
         return xmlProperties;
     }
 
@@ -92,12 +89,14 @@
         return cron;
     }
 
-    public List getPinnedServers() {
+    public List<String> getPinnedServers() {
         return pinnedServers;
     }
 
-    public void setPinnedServers(List pinnedServers) {
-        this.pinnedServers = pinnedServers;
+    public void setPinnedServers(List<String> pinnedServers) {
+        if (pinnedServers != null) {
+            this.pinnedServers.addAll(pinnedServers);
+        }
     }
 
     public String getName() {

Modified: 
synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task/TaskDescriptionFactory.java
URL: 
http://svn.apache.org/viewvc/synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task/TaskDescriptionFactory.java?rev=712705&r1=712704&r2=712705&view=diff
==============================================================================
--- 
synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task/TaskDescriptionFactory.java
 (original)
+++ 
synapse/trunk/java/modules/tasks/src/main/java/org/apache/synapse/task/TaskDescriptionFactory.java
 Mon Nov 10 09:37:39 2008
@@ -83,7 +83,7 @@
                     // default to all servers
                 } else {
                     StringTokenizer st = new 
StringTokenizer(pinnedServersValue, " ,");
-                    List pinnedServersList = new ArrayList();
+                    List<String> pinnedServersList = new ArrayList<String>();
                     while (st.hasMoreTokens()) {
                         String token = st.nextToken();
                         if (token.length() != 0) {