Author: lahiru
Date: Mon Aug 26 21:08:31 2013
New Revision: 1517698

URL: http://svn.apache.org/r1517698
Log:
fixing in https://issues.apache.org/jira/browse/AIRAVATA-779.

Modified:
    
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/GFacAPI.java
    
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/handler/GridFTPInputHandler.java
    
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/utils/GramRSLGenerator.java
    
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/utils/OutputUtils.java

Modified: 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/GFacAPI.java
URL: 
http://svn.apache.org/viewvc/airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/GFacAPI.java?rev=1517698&r1=1517697&r2=1517698&view=diff
==============================================================================
--- 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/GFacAPI.java
 (original)
+++ 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/GFacAPI.java
 Mon Aug 26 21:08:31 2013
@@ -77,7 +77,12 @@ public class GFacAPI {
             jobExecutionContext.setProperty(ERROR_SENT,"true");
             jobExecutionContext.getNotifier().publish(new 
ExecutionFailEvent(e.getCause()));
             throw e;
-        } finally {
+        } catch (Exception e){
+            jobExecutionContext.setProperty(ERROR_SENT,"true");
+            jobExecutionContext.getNotifier().publish(new 
ExecutionFailEvent(e.getCause()));
+            throw new GFacException(e);
+        }
+        finally {
             try{
             invokeOutFlowHandlers(jobExecutionContext);
             }catch(GFacException e){

Modified: 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/handler/GridFTPInputHandler.java
URL: 
http://svn.apache.org/viewvc/airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/handler/GridFTPInputHandler.java?rev=1517698&r1=1517697&r2=1517698&view=diff
==============================================================================
--- 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/handler/GridFTPInputHandler.java
 (original)
+++ 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/handler/GridFTPInputHandler.java
 Mon Aug 26 21:08:31 2013
@@ -53,8 +53,6 @@ public class GridFTPInputHandler impleme
 
     public void invoke(JobExecutionContext jobExecutionContext) throws 
GFacHandlerException {
         log.info("Invoking GridFTPInputHandler ...");
-
-
         MessageContext inputNew = new MessageContext();
         try {
             MessageContext input = jobExecutionContext.getInMessageContext();

Modified: 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/utils/GramRSLGenerator.java
URL: 
http://svn.apache.org/viewvc/airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/utils/GramRSLGenerator.java?rev=1517698&r1=1517697&r2=1517698&view=diff
==============================================================================
--- 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/utils/GramRSLGenerator.java
 (original)
+++ 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/utils/GramRSLGenerator.java
 Mon Aug 26 21:08:31 2013
@@ -20,6 +20,7 @@
 */
 package org.apache.airavata.gfac.utils;
 
+import org.apache.airavata.common.utils.StringUtil;
 import org.apache.airavata.commons.gfac.type.ActualParameter;
 import org.apache.airavata.commons.gfac.type.MappingFactory;
 import org.apache.airavata.gfac.Constants;
@@ -27,10 +28,7 @@ import org.apache.airavata.gfac.ToolsExc
 import org.apache.airavata.gfac.context.JobExecutionContext;
 import org.apache.airavata.gfac.context.MessageContext;
 import org.apache.airavata.gfac.provider.GFacProviderException;
-import org.apache.airavata.schemas.gfac.HpcApplicationDeploymentType;
-import org.apache.airavata.schemas.gfac.NameValuePairType;
-import org.apache.airavata.schemas.gfac.QueueType;
-import org.apache.airavata.schemas.gfac.URIArrayType;
+import org.apache.airavata.schemas.gfac.*;
 import org.apache.airavata.schemas.wec.ContextHeaderDocument;
 import org.globus.gram.GramAttributes;
 import org.slf4j.Logger;
@@ -49,7 +47,7 @@ public class GramRSLGenerator {
 
     ;
 
-    public static GramAttributes configureRemoteJob(JobExecutionContext 
context) throws ToolsException {
+    public static GramAttributes  configureRemoteJob(JobExecutionContext 
context) throws ToolsException {
         HpcApplicationDeploymentType app = (HpcApplicationDeploymentType) 
context.getApplicationContext().getApplicationDeploymentDescription().getType();
         GramAttributes jobAttr = new GramAttributes();
         jobAttr.setExecutable(app.getExecutableLocation());
@@ -86,11 +84,18 @@ public class GramRSLGenerator {
             Set<String> keys = inputs.keySet();
             for (String paramName : keys ) {
                ActualParameter actualParameter = (ActualParameter) 
inputs.get(paramName);
-                if 
("URIArray".equals(actualParameter.getType().getType().toString())) {
-                    String[] values = ((URIArrayType) 
actualParameter.getType()).getValueArray();
-                    for (String value : values) {
-                        jobAttr.addArgument(value);
+                if 
("URIArray".equals(actualParameter.getType().getType().toString()) || 
"StringArray".equals(actualParameter.getType().getType().toString())
+                        || 
"FileArray".equals(actualParameter.getType().getType().toString())) {
+                    String[] values = null;
+                    if (actualParameter.getType() instanceof URIArrayType) {
+                        values = ((URIArrayType) 
actualParameter.getType()).getValueArray();
+                    } else if (actualParameter.getType() instanceof 
StringArrayType) {
+                        values = ((StringArrayType) 
actualParameter.getType()).getValueArray();
+                    } else if (actualParameter.getType() instanceof 
FileArrayType) {
+                        values = ((FileArrayType) 
actualParameter.getType()).getValueArray();
                     }
+                    String value = StringUtil.createDelimiteredString(values, 
" ");
+                    jobAttr.addArgument(value);
                 } else {
                     String paramValue = 
MappingFactory.toString(actualParameter);
                     jobAttr.addArgument(paramValue);

Modified: 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/utils/OutputUtils.java
URL: 
http://svn.apache.org/viewvc/airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/utils/OutputUtils.java?rev=1517698&r1=1517697&r2=1517698&view=diff
==============================================================================
--- 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/utils/OutputUtils.java
 (original)
+++ 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/utils/OutputUtils.java
 Mon Aug 26 21:08:31 2013
@@ -59,9 +59,6 @@ public class OutputUtils {
                 ((StdErrParameterType) actual.getType()).setValue(stderr);
                 result.put(paramName, actual);
             } else {
-               if ("URI".equals(actual.getType().getType().toString()) &&  
!((URIParameterType) actual.getType()).getValue().isEmpty()){
-                       continue;
-               }
                 String parseStdout = parseStdout(stdout, paramName);
                 if (parseStdout != null) {
                     MappingFactory.fromString(actual, parseStdout);


Reply via email to