Mike Kolesnik has submitted this change and it was merged. Change subject: core: Improve exception handling in end actions ......................................................................
core: Improve exception handling in end actions The end action didn't handle exceptions well, always trying to clean up the compensation if transaction was still active. However, if an exception had occurred in the end action, then actually it would be a good idea to compensate since rollback will follow. Also added a try catch for other cases when exception during the compensation can cause problems. Change-Id: I40f6c12512c5b71b16c1de0d97a509848e8a4beb --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 26 insertions(+), 5 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2446 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40f6c12512c5b71b16c1de0d97a509848e8a4beb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkole...@redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
