Tomas Jelinek has posted comments on this change. Change subject: userportal,webadmin: implement VM builder ......................................................................
Patch Set 9: sorry for not answering also here - we have discussed this on IRC. The specific proposal was, that there would be a decorator for each entity having the same interface. Than, you could have only one builder and switch the FE-BE models. The problem here is that: instead of 2 builders you would have to maintain 2 decorators + 1 builder. But the most important, you have different expectations from the FE->BE builders than the opposite way around (e.g. if need to do some server call to build up the entity) And this does not contain all the possible builders, this is just a first batch - some others will follow... -- To view, visit http://gerrit.ovirt.org/13915 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I54d938896e469c73800297c3d66185a439243a41 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <[email protected]> Gerrit-Reviewer: Alona Kaplan <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Tomas Jelinek <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
