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@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@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Martin Peřina <mperina@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: mooli tayer <mtayer@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server
participants (1)
-
emesika@redhat.com