You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oodt.apache.org by bf...@apache.org on 2011/03/09 00:12:22 UTC

svn commit: r1079597 - /oodt/branches/wengine-branch/wengine/src/main/java/org/apache/oodt/cas/workflow/instance/WorkflowConnectTaskInstance.java

Author: bfoster
Date: Tue Mar  8 23:12:21 2011
New Revision: 1079597

URL: http://svn.apache.org/viewvc?rev=1079597&view=rev
Log:

- improvements to ConnectWorkflowTaskInstance

--------------------

Modified:
    oodt/branches/wengine-branch/wengine/src/main/java/org/apache/oodt/cas/workflow/instance/WorkflowConnectTaskInstance.java

Modified: oodt/branches/wengine-branch/wengine/src/main/java/org/apache/oodt/cas/workflow/instance/WorkflowConnectTaskInstance.java
URL: http://svn.apache.org/viewvc/oodt/branches/wengine-branch/wengine/src/main/java/org/apache/oodt/cas/workflow/instance/WorkflowConnectTaskInstance.java?rev=1079597&r1=1079596&r2=1079597&view=diff
==============================================================================
--- oodt/branches/wengine-branch/wengine/src/main/java/org/apache/oodt/cas/workflow/instance/WorkflowConnectTaskInstance.java (original)
+++ oodt/branches/wengine-branch/wengine/src/main/java/org/apache/oodt/cas/workflow/instance/WorkflowConnectTaskInstance.java Tue Mar  8 23:12:21 2011
@@ -73,14 +73,6 @@ public class WorkflowConnectTaskInstance
 	protected ResultsState performExecution(ControlMetadata ctrlMetadata) {
 		if (ctrlMetadata.getMetadata(SPAWNED_WORKFLOWS) == null) {
 			
-			//Add spawning keys to localized workflow metadata keys
-			Vector<String> localKeys = new Vector<String>();
-			if (ctrlMetadata.getMetadata(WorkflowProcessor.LOCAL_KEYS) != null)
-				localKeys.addAll(ctrlMetadata.getAllMetadata(WorkflowProcessor.LOCAL_KEYS));
-			localKeys.addAll(Arrays.asList(SPAWNED_WORKFLOWS, SPAWNED_BY_WORKFLOW));
-			ctrlMetadata.replaceLocalMetadata(WorkflowProcessor.LOCAL_KEYS, localKeys);
-			ctrlMetadata.setAsWorkflowMetadataKey(WorkflowProcessor.LOCAL_KEYS);
-			
 			//Get Spawn ModelId
 			String spawnModelId = ctrlMetadata.getMetadata(SPAWN_MODEL_ID);
 			if (spawnModelId == null)
@@ -119,7 +111,7 @@ public class WorkflowConnectTaskInstance
 				LOG.log(Level.SEVERE, "Failed to determine N : " + e.getMessage(), e);
 				return new ResultsFailureState("Failed to determine N : " + e.getMessage());
 			}
-			Metadata spawnWorkflowMet = ctrlMetadata.asMetadata(ControlMetadata.DYN);
+			Metadata spawnWorkflowMet = ctrlMetadata.asMetadata(ControlMetadata.LOCAL, ControlMetadata.DYN);
 			Vector<String> spawnedInstanceIds = new Vector<String>();
 			for (int i = 0; i < n; i++) {
 				Metadata curWorkflowMet = new Metadata(spawnWorkflowMet);
@@ -143,6 +135,14 @@ public class WorkflowConnectTaskInstance
 			ctrlMetadata.replaceLocalMetadata(SPAWNED_WORKFLOWS, spawnedInstanceIds);
 			ctrlMetadata.setAsWorkflowMetadataKey(SPAWNED_WORKFLOWS);
 			
+			//Add spawning keys to localized workflow metadata keys
+			Vector<String> localKeys = new Vector<String>();
+			if (ctrlMetadata.getMetadata(WorkflowProcessor.LOCAL_KEYS) != null)
+				localKeys.addAll(ctrlMetadata.getAllMetadata(WorkflowProcessor.LOCAL_KEYS));
+			localKeys.addAll(Arrays.asList(SPAWNED_WORKFLOWS, SPAWNED_BY_WORKFLOW));
+			ctrlMetadata.replaceLocalMetadata(WorkflowProcessor.LOCAL_KEYS, localKeys);
+			ctrlMetadata.setAsWorkflowMetadataKey(WorkflowProcessor.LOCAL_KEYS);
+			
 			return new ResultsBailState("Waiting for " + n + " of " + n + " spawned workflows to complete");
 		}else {