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 <amureini@redhat.com> --- 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 <amureini@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Ayal Baron <abaron@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra@linux.vnet.ibm.com>