You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airavata.apache.org by sh...@apache.org on 2014/11/05 21:06:07 UTC

git commit: Changed job submission and data movement protocol to their enums instead of String which will cause to NPE with null values

Repository: airavata
Updated Branches:
  refs/heads/gfac_appcatalog_int ce1354e96 -> 3eea1a309


Changed job submission and data movement protocol to their enums instead of String which will cause to NPE with null values


Project: http://git-wip-us.apache.org/repos/asf/airavata/repo
Commit: http://git-wip-us.apache.org/repos/asf/airavata/commit/3eea1a30
Tree: http://git-wip-us.apache.org/repos/asf/airavata/tree/3eea1a30
Diff: http://git-wip-us.apache.org/repos/asf/airavata/diff/3eea1a30

Branch: refs/heads/gfac_appcatalog_int
Commit: 3eea1a3095f735f7db3b37e65f92b653f0f28f03
Parents: ce1354e
Author: shamrath <sh...@gmail.com>
Authored: Wed Nov 5 15:05:37 2014 -0500
Committer: shamrath <sh...@gmail.com>
Committed: Wed Nov 5 15:05:37 2014 -0500

----------------------------------------------------------------------
 .../client/tools/RegisterSampleApplicationsUtils.java        | 7 ++++---
 .../airavata/integration/tools/DocumentCreatorUtils.java     | 8 ++++----
 2 files changed, 8 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/airavata/blob/3eea1a30/airavata-api/airavata-client-sdks/java-client-samples/src/main/java/org/apache/airavata/client/tools/RegisterSampleApplicationsUtils.java
----------------------------------------------------------------------
diff --git a/airavata-api/airavata-client-sdks/java-client-samples/src/main/java/org/apache/airavata/client/tools/RegisterSampleApplicationsUtils.java b/airavata-api/airavata-client-sdks/java-client-samples/src/main/java/org/apache/airavata/client/tools/RegisterSampleApplicationsUtils.java
index 24c0688..184f3e9 100644
--- a/airavata-api/airavata-client-sdks/java-client-samples/src/main/java/org/apache/airavata/client/tools/RegisterSampleApplicationsUtils.java
+++ b/airavata-api/airavata-client-sdks/java-client-samples/src/main/java/org/apache/airavata/client/tools/RegisterSampleApplicationsUtils.java
@@ -41,15 +41,16 @@ public class RegisterSampleApplicationsUtils {
     public static ComputeResourcePreference
         createComputeResourcePreference(String computeResourceId, String allocationProjectNumber,
                                         boolean overridebyAiravata, String preferredBatchQueue,
-                                        String preferredJobSubmissionProtocol,String preferredDataMovementProtocol,
+                                        JobSubmissionProtocol preferredJobSubmissionProtocol,
+                                        DataMovementProtocol preferredDataMovementProtocol,
                                         String scratchLocation) {
         ComputeResourcePreference computeResourcePreference = new ComputeResourcePreference();
         computeResourcePreference.setComputeResourceId(computeResourceId);
         computeResourcePreference.setOverridebyAiravata(overridebyAiravata);
         computeResourcePreference.setAllocationProjectNumber(allocationProjectNumber);
         computeResourcePreference.setPreferredBatchQueue(preferredBatchQueue);
-        computeResourcePreference.setPreferredDataMovementProtocol(DataMovementProtocol.valueOf(preferredDataMovementProtocol));
-        computeResourcePreference.setPreferredJobSubmissionProtocol(JobSubmissionProtocol.valueOf(preferredJobSubmissionProtocol));
+        computeResourcePreference.setPreferredDataMovementProtocol(preferredDataMovementProtocol);
+        computeResourcePreference.setPreferredJobSubmissionProtocol(preferredJobSubmissionProtocol);
         computeResourcePreference.setScratchLocation(scratchLocation);
         return computeResourcePreference;
     }

http://git-wip-us.apache.org/repos/asf/airavata/blob/3eea1a30/modules/integration-tests/src/test/java/org/apache/airavata/integration/tools/DocumentCreatorUtils.java
----------------------------------------------------------------------
diff --git a/modules/integration-tests/src/test/java/org/apache/airavata/integration/tools/DocumentCreatorUtils.java b/modules/integration-tests/src/test/java/org/apache/airavata/integration/tools/DocumentCreatorUtils.java
index 2fc3ac2..a49d22f 100644
--- a/modules/integration-tests/src/test/java/org/apache/airavata/integration/tools/DocumentCreatorUtils.java
+++ b/modules/integration-tests/src/test/java/org/apache/airavata/integration/tools/DocumentCreatorUtils.java
@@ -45,15 +45,15 @@ public class DocumentCreatorUtils {
 
 	public static ComputeResourcePreference createComputeResourcePreference(String computeResourceId, String scratchLocation,
 			String allocationProjectNumber, boolean overridebyAiravata,
-			String preferredBatchQueue, String preferredJobSubmissionProtocol,
-			String preferredDataMovementProtocol) throws AppCatalogException {
+			String preferredBatchQueue, JobSubmissionProtocol preferredJobSubmissionProtocol,
+			DataMovementProtocol preferredDataMovementProtocol) throws AppCatalogException {
 		ComputeResourcePreference computeResourcePreference = new ComputeResourcePreference();
 		computeResourcePreference.setComputeResourceId(computeResourceId);
 		computeResourcePreference.setOverridebyAiravata(overridebyAiravata);
 		computeResourcePreference.setAllocationProjectNumber(allocationProjectNumber);
 		computeResourcePreference.setPreferredBatchQueue(preferredBatchQueue);
-		computeResourcePreference.setPreferredDataMovementProtocol(DataMovementProtocol.valueOf(preferredDataMovementProtocol));
-		computeResourcePreference.setPreferredJobSubmissionProtocol(JobSubmissionProtocol.valueOf(preferredJobSubmissionProtocol));
+		computeResourcePreference.setPreferredDataMovementProtocol(preferredDataMovementProtocol);
+		computeResourcePreference.setPreferredJobSubmissionProtocol(preferredJobSubmissionProtocol);
 		computeResourcePreference.setScratchLocation(scratchLocation);
 		return computeResourcePreference;
 	}