Michael Kublin has posted comments on this change.

Change subject: core: AddVdsSpmId - execute for a number of VDSs at once.
......................................................................


Patch Set 3: Fails; Do not submit

Michael, can you elaborate what is complicated in the code? - I know only the 
one way, I will write code instead of you, with less number of lines and with 
out using all data structure that you used. 
The patch is irrelevant and will not work, because of regression. That should 
be fixed at http://gerrit.ovirt.org/#/c/11271/.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9ecf9990eeda676ad2a4f9bd73d4535bd2eb3f9a
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to