You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airavata.apache.org by la...@apache.org on 2016/09/27 07:28:43 UTC

airavata git commit: String.format

Repository: airavata
Updated Branches:
  refs/heads/lahiru/AIRAVATA-2065 1211aa484 -> b19b0edb7


String.format


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

Branch: refs/heads/lahiru/AIRAVATA-2065
Commit: b19b0edb741b626a89e88c57a4d29c9cb3121d82
Parents: 1211aa4
Author: Lahiru Ginnaliya Gamathige <la...@apache.org>
Authored: Tue Sep 27 00:28:33 2016 -0700
Committer: Lahiru Ginnaliya Gamathige <la...@apache.org>
Committed: Tue Sep 27 00:28:33 2016 -0700

----------------------------------------------------------------------
 .../airavata/gfac/monitor/email/EmailBasedMonitor.java    | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/airavata/blob/b19b0edb/modules/gfac/gfac-impl/src/main/java/org/apache/airavata/gfac/monitor/email/EmailBasedMonitor.java
----------------------------------------------------------------------
diff --git a/modules/gfac/gfac-impl/src/main/java/org/apache/airavata/gfac/monitor/email/EmailBasedMonitor.java b/modules/gfac/gfac-impl/src/main/java/org/apache/airavata/gfac/monitor/email/EmailBasedMonitor.java
index bbcd635..ab518f3 100644
--- a/modules/gfac/gfac-impl/src/main/java/org/apache/airavata/gfac/monitor/email/EmailBasedMonitor.java
+++ b/modules/gfac/gfac-impl/src/main/java/org/apache/airavata/gfac/monitor/email/EmailBasedMonitor.java
@@ -203,7 +203,7 @@ public class EmailBasedMonitor implements JobMonitor, Runnable{
 					    continue;
 				    } else {
                         quite = false;
-					    log.info("[EJM]: {} job/s in job monitor map", jobMonitorMap.size());
+					    log.info(String.format("[EJM]: %d job/s in job monitor map", jobMonitorMap.size()));
 				    }
 				    if (!store.isConnected()) {
 					    store.connect();
@@ -267,13 +267,13 @@ public class EmailBasedMonitor implements JobMonitor, Runnable{
                 if (jobStatusResult.getJobId() != null) {
                     taskContext = jobMonitorMap.get(jobStatusResult.getJobId());
                 } else {
-                    log.info("Returned null for job id, message subject--> {}" , message.getSubject());
+                    log.info(String.format("Returned null for job id, message subject--> %s" , message.getSubject()));
                 }
                 if (taskContext == null) {
                     if (jobStatusResult.getJobName() != null) {
                         taskContext = jobMonitorMap.get(jobStatusResult.getJobName());
                     } else {
-                        log.info("Returned null for job name, message subject --> {}" , message.getSubject());
+                        log.info(String.format("Returned null for job name, message subject --> %s" , message.getSubject()));
                     }
                 }
                 if (taskContext != null) {
@@ -378,9 +378,9 @@ public class EmailBasedMonitor implements JobMonitor, Runnable{
 			    log.info("[EJM]: Publishing status changes to amqp. " + jobDetails);
 			    GFacUtils.saveJobStatus(parentProcessContext, jobModel);
 		    } catch (GFacException e) {
-			    log.error("expId: {}, processId: {}, taskId: {}, jobId: {} :- Error while save and publishing Job " +
+			    log.error(String.format("expId: %s, processId: %s, taskId: %s, jobId: %s :- Error while save and publishing Job " +
                         "status {}", taskContext.getExperimentId(), taskContext.getProcessId(), jobModel
-                        .getTaskId(), jobModel.getJobId(), jobStatus.getJobState());
+                        .getTaskId(), jobModel.getJobId(), jobStatus.getJobState()));
 		    }
 	    }