Author: raminder
Date: Fri Feb 15 18:35:27 2013
New Revision: 1446717

URL: http://svn.apache.org/r1446717
Log:
Removed the job cancel part from finally. We don't want to cancel the jobs on 
gram errors. There can be communication error with gram server but PBS job need 
not to be cancel. AIRAVATA-764

Modified:
    
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/provider/impl/GramProvider.java

Modified: 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/provider/impl/GramProvider.java
URL: 
http://svn.apache.org/viewvc/airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/provider/impl/GramProvider.java?rev=1446717&r1=1446716&r2=1446717&view=diff
==============================================================================
--- 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/provider/impl/GramProvider.java
 (original)
+++ 
airavata/trunk/modules/gfac-core/src/main/java/org/apache/airavata/gfac/provider/impl/GramProvider.java
 Fri Feb 15 18:35:27 2013
@@ -93,10 +93,6 @@ public class GramProvider implements GFa
             */
             listener.waitFor();
 
-            /*
-            * Remove listener
-            */
-            job.removeListener(listener);
 
             /*
             * Fail job
@@ -128,8 +124,12 @@ public class GramProvider implements GFa
         } finally {
             if (job != null) {
                 try {
-                    job.cancel();
+                        /*
+                     * Remove listener
+                     */
+                     job.removeListener(listener);
                 } catch (Exception e) {
+                        log.error(e.getMessage());
                 }
             }
         }


Reply via email to