Change in ovirt-engine[master]: core, restapi: Replaced system stats query fields with const...

tnisan at redhat.com tnisan at redhat.com
Mon Feb 18 11:56:27 UTC 2013


Tal Nisan has submitted this change and it was merged.

Change subject: core, restapi: Replaced system stats query fields with constants
......................................................................


core, restapi: Replaced system stats query fields with constants

Replaced the hard coded String fields returning in GetSystemStatistics
query to constants, also changed the call to this query in the REST to use
the contstants when parsing the result

Change-Id: I1a2e3fc22c4c8228ec9362d1fb2a608c4db4746c
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetSystemStatisticsQuery.java
A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/constants/QueryConstants.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java
3 files changed, 40 insertions(+), 19 deletions(-)

Approvals:
  Tal Nisan: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/12143
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1a2e3fc22c4c8228ec9362d1fb2a608c4db4746c
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>



More information about the Engine-commits mailing list