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 <[email protected]> Authored: Tue Dec 6 12:59:23 2016 -0600 Committer: Joshua Cohen <[email protected]> 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; } }
