Change in ovirt-engine[master]: notifier: Use Runtime.addShutdownHook to catch shutdown signals

emesika at redhat.com emesika at redhat.com
Mon Aug 19 08:28:27 UTC 2013


Eli Mesika has submitted this change and it was merged.

Change subject: notifier: Use Runtime.addShutdownHook to catch shutdown signals
......................................................................


notifier: Use Runtime.addShutdownHook to catch shutdown signals

Use Runtime.addShutdownHook to catch process shutdown
(SIGTERM, SIGINT, SIGHUP) and not sun's proprietary
sun.misc package. Runtime.addShutdownHook is JVM
agnostic and has the advantage of executing the same
if the service was killed or the java process it self.

Change-Id: I71b3a2a54862b9c3ca1fb6d2c32f127b833953de
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=749513
Signed-off-by: Mooli Tayer <mtayer at redhat.com>
---
M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/Notifier.java
1 file changed, 5 insertions(+), 7 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I71b3a2a54862b9c3ca1fb6d2c32f127b833953de
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list