Tal Nisan has posted comments on this change.

Change subject: core: unify and move Disk Comparators
......................................................................


Patch Set 4:

Sorry, I copy pasted from the wrong place, I meant putting them back in 
core.vdsbroker
Like patch set 2, the idea of putting them in businessentities was in order for 
them to be accessible in the frontend, since we don't use it there and a 
comparator is not a business entity we should probably move them back to 
core.vdsbroker

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ide5808ba4404d141f43911945e19f170e9694da4
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Alissa Bonas <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Liron Ar <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Sergey Gotliv <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to