Change in ovirt-engine[master]: notifier: Use Runtime.addShutdownHook to catch shutdown signals
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(a)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(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server