Maor Lipchuk has posted comments on this change.

Change subject: core: Add MoveImageGroup to MoveImageGroupCommand
......................................................................


Patch Set 3:

@Liron
Move operation is always being called with copy collapse=false so we will 
always call MoveImageGroupCommand, I'm not sure that we can support move 
operation with copy collapse=true, but when we decide that we do, then we can 
change the code accordingly.

the motivation is that using copy elements in the move operations seems 
confusing IMHO to read and understand the code

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf34e2b6c3c33dd86fe4b4243c86b01cd1b19f7c
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Liron Ar <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to