Change in ovirt-engine[master]: core: BaseImagesCommand: Cleanup RemoveSnapshot

mlipchuk at redhat.com mlipchuk at redhat.com
Mon Jul 23 15:05:53 UTC 2012


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



More information about the Engine-commits mailing list