Merge pull request #902 from vorburger/JobLogImprovementIllustratingBug
JobRegisterServiceImpl error logging improvement
This commit is contained in:
+9
-2
@@ -108,8 +108,12 @@ public class JobRegisterServiceImpl implements JobRegisterService {
|
|||||||
}
|
}
|
||||||
|
|
||||||
} catch (final Exception e) {
|
} catch (final Exception e) {
|
||||||
throw new PlatformInternalServerException("error.msg.sheduler.job.execution.failed", "Job execution failed for job with id:"
|
final String msg = "Job execution failed for job with id:"
|
||||||
+ scheduledJobDetail.getId(), scheduledJobDetail.getId());
|
+ scheduledJobDetail.getId();
|
||||||
|
logger.error(msg, e);
|
||||||
|
throw new PlatformInternalServerException(
|
||||||
|
"error.msg.sheduler.job.execution.failed", msg,
|
||||||
|
scheduledJobDetail.getId());
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
@@ -214,6 +218,9 @@ public class JobRegisterServiceImpl implements JobRegisterService {
|
|||||||
scheduledJobDetails.updateNextRunTime(null);
|
scheduledJobDetails.updateNextRunTime(null);
|
||||||
final String stackTrace = getStackTraceAsString(throwable);
|
final String stackTrace = getStackTraceAsString(throwable);
|
||||||
scheduledJobDetails.updateErrorLog(stackTrace);
|
scheduledJobDetails.updateErrorLog(stackTrace);
|
||||||
|
logger.error(
|
||||||
|
"Could not schedule job: "
|
||||||
|
+ scheduledJobDetails.getJobName(), throwable);
|
||||||
}
|
}
|
||||||
scheduledJobDetails.updateCurrentlyRunningStatus(false);
|
scheduledJobDetails.updateCurrentlyRunningStatus(false);
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user