Arik Hadas has posted comments on this change.

Change subject: core: remove memory snapshot when its not needed
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/SnapshotDaoDbFacadeImpl.java
Line 223:                 parameterSource);
Line 224:     }
Line 225: 
Line 226:     @Override
Line 227:     public void removeMemoryFromSnapshot(Guid snapshotId) {
Done
Line 228:         MapSqlParameterSource parameterSource = 
getCustomMapSqlParameterSource()
Line 229:                 .addValue("snapshot_id", snapshotId)
Line 230:                 .addValue("memory_volume", null);
Line 231: 


-- 
To view, visit http://gerrit.ovirt.org/15682
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic087580475cd4fcfac969ddf0f69929fa247f86d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to