Alon Bar-Lev has posted comments on this change. Change subject: webadmin: Use ConfigCommon version in AsyncDataProvider ......................................................................
Patch Set 1: I would have split it based on executable units. The GWT is a remote application interacting with engine, had we done this better, we would have used the restapi to do the communication. The fact that it is maintained at same tree as engine is something I would like to see gone in future :) The rest api is part of the backend, it is not standalone execution unit, right? -- To view, visit http://gerrit.ovirt.org/11174 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4c9213aee9182d3bb27fbba04f7163b502438a3e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <aw...@redhat.com> Gerrit-Reviewer: Alexander Wels <aw...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Einav Cohen <eco...@redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches