Change in ovirt-engine[master]: core: Fix QuartzException when Ctrl+C engine

oourfali at redhat.com oourfali at redhat.com
Tue Aug 12 06:49:09 UTC 2014


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: lzelkha at redhat.com <lzelkha at redhat.com>
---
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 <lzelkha at redhat.com>
Gerrit-Reviewer: Eli Mesika <elimesika at gmail.com>
Gerrit-Reviewer: Liran Zelkha <lzelkha at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list