Allon Mureinik has posted comments on this change. Change subject: core: Validate maximum number of hosts in DC.(#771699) ......................................................................
Patch Set 4: Yair: regarding DAO tests - i somewhat disagree. You'll be introducing a dead-code patch - a new method in the DAO, that although used, is never called, and is essentially useless (in thaat patch's context). This should be reserved for the (hopefully rare) occasions that the BLL patch is so humongous it's impossible to review it together with the DAO change. -- To view, visit http://gerrit.ovirt.org/8321 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If375400f3e12e3e0452053dea12ac6e28bc0ff61 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Michael Kublin <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
