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

Reply via email to