Asaf Shakarchi has posted comments on this change. Change subject: WIP: working CDI solution for backend ......................................................................
Patch Set 4: Adding myself into this: This is good stuff, I personally would love to see DI/IOC inside backend (btw, I pushed DI via GIN into our client code architecture from the first day.) And while knowing the competitor of CDI (Guice) pretty well, before integration (although it smells so good to go with Weld which ships with AS7 out of the box) i'd like to ensure we will cover through this approach: - Plugins architecture for external components, with class loading isolation and security? - Is it going to work without EJB container without much hassle? I'd love to see backend independent of EJB as we anyway don't leverage its power anyway (but this discussion is for another thread) - I'd like to extends the concepts of DI in other levels such as REST API (at least on a wiki doc) if/how we'r going to use this on the API level, for instance "injection" is maintained manually in few places. (e.g AbstractBackendResource#inject(S resource)) Thanks. -- To view, visit http://gerrit.ovirt.org/5575 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7f604ff91847b698efe84a09f724ba0492a672c1 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <[email protected]> Gerrit-Reviewer: Asaf Shakarchi <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Laszlo Hornyak <[email protected]> Gerrit-Reviewer: Michael Kublin <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
