Michael Kublin has posted comments on this change.

Change subject: core: ImportVmCommand - added method getUpdatedVmForOperation()
......................................................................


Patch Set 3:

"mkublin: even though these changes are on the same file, they aren't 
[necessarily] related. Each one has it's own merit, and separating them makes 
them easier to understand, review and cherry-pick if required." - If it not 
related they should not be at the same patch set, and not depends on each 
other. Also if they at the same patch set they can not be merged separately, so 
you will not achieve your goal

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1f53af1324fc1aed32bdf8ec1eac9b7f7d307f25
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
Gerrit-Reviewer: Sharad Mishra <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to