Maor Lipchuk has submitted this change and it was merged. Change subject: core: BaseImagesCommand: Cleanup RemoveSnapshot ......................................................................
core: BaseImagesCommand: Cleanup RemoveSnapshot The RemoveSnapshot method was made up from the RemoveSnapshotFromDB and AdditionalImageRemoveTreatment methods. AdditionalImageRemoveTreatment was only called from RestoreFromSnapshotCommand, so the additional logic was moved to the overriding method there. Change-Id: I36c6ffe9b17e8ff65e99c8b715cd9a30224e10ab Signed-off-by: Allon Mureinik <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestoreFromSnapshotCommand.java 2 files changed, 12 insertions(+), 21 deletions(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6400 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I36c6ffe9b17e8ff65e99c8b715cd9a30224e10ab Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Sharad Mishra <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
