Oved Ourfali has submitted this change and it was merged.

Change subject: core: Fix QuartzException when Ctrl+C engine
......................................................................


core: Fix QuartzException when Ctrl+C engine

When shuting down engine in dev environment using Ctrl+C
exceptions are printed to the screen because of Quartz shutdown.
This patch validates that the scheduler can indeed reschedule a job before 
rescheduling it.

Exception Dump is:
failed to reschedule the job: org.quartz.SchedulerException: The Scheduler has 
been shutdown.
at org.quartz.core.QuartzScheduler.validateState(QuartzScheduler.java:749) 
[quartz.jar:]
at org.quartz.core.QuartzScheduler.rescheduleJob(QuartzScheduler.java:1060) 
[quartz.jar:]
at org.quartz.impl.StdScheduler.rescheduleJob(StdScheduler.java:312) 
[quartz.jar:]
at 
org.ovirt.engine.core.utils.timer.SchedulerUtilQuartzImpl.rescheduleAJob(SchedulerUtilQuartzImpl.java:331)
 [scheduler.jar:]
at 
org.ovirt.engine.core.utils.timer.FixedDelayJobListener.jobWasExecuted(FixedDelayJobListener.java:91)
 [scheduler.jar:]

Change-Id: I3fcdc3aa51bbcdf817363af6431db488501aa401
Signed-off-by: [email protected] <[email protected]>
---
M 
backend/manager/modules/scheduler/src/main/java/org/ovirt/engine/core/utils/timer/SchedulerUtilQuartzImpl.java
1 file changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Liran Zelkha: Verified
  Yair Zaslavsky: Looks good to me, but someone else must approve
  Oved Ourfali: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/31310
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3fcdc3aa51bbcdf817363af6431db488501aa401
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to