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