You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@aurora.apache.org by jc...@apache.org on 2016/12/06 18:59:45 UTC
aurora git commit: Fix invalid logging that was causing pmd to NPE.
Repository: aurora
Updated Branches:
refs/heads/master 4bc524614 -> 91ddb075c
Fix invalid logging that was causing pmd to NPE.
Reviewed at https://reviews.apache.org/r/54428/
Project: http://git-wip-us.apache.org/repos/asf/aurora/repo
Commit: http://git-wip-us.apache.org/repos/asf/aurora/commit/91ddb075
Tree: http://git-wip-us.apache.org/repos/asf/aurora/tree/91ddb075
Diff: http://git-wip-us.apache.org/repos/asf/aurora/diff/91ddb075
Branch: refs/heads/master
Commit: 91ddb075cf37cc15cee9c2b15cff0f71a950d551
Parents: 4bc5246
Author: Joshua Cohen <jc...@apache.org>
Authored: Tue Dec 6 12:59:23 2016 -0600
Committer: Joshua Cohen <jc...@apache.org>
Committed: Tue Dec 6 12:59:23 2016 -0600
----------------------------------------------------------------------
.../org/apache/aurora/scheduler/cron/quartz/AuroraCronJob.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/aurora/blob/91ddb075/src/main/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJob.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJob.java b/src/main/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJob.java
index ae7d019..5873983 100644
--- a/src/main/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJob.java
+++ b/src/main/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJob.java
@@ -154,9 +154,9 @@ class AuroraCronJob implements Job, EventSubscriber {
if (killFollowups.contains(key)) {
context.getJobDetail().getJobDataMap().remove(path);
killFollowups.remove(key);
- LOG.info("Resetting job context for cron " + path);
+ LOG.info("Resetting job context for cron {}", path);
} else {
- LOG.info("Ignoring trigger as another concurrent run is active for cron " + path);
+ LOG.info("Ignoring trigger as another concurrent run is active for cron {}", path);
return;
}
}