Change in ovirt-engine[master]: core: Decrease updates to VdsDynamic

oourfali at redhat.com oourfali at redhat.com
Thu Sep 4 07:21:18 UTC 2014


Oved Ourfali has submitted this change and it was merged.

Change subject: core: Decrease updates to VdsDynamic
......................................................................


core: Decrease updates to VdsDynamic

VdsDynamic updates are heavy on the database. This patch minimizes
these writes by fixing some issues with VdsDynamic equals method
and improving the RowMapper of VdsDynamic to include all fields. 
A future patch will move cpu_over_
commitment to VdsStatistics

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1136345
Change-Id: I30711bc02adcc6ebbcac95d609da0fbc54f49cb2
Signed-off-by: lzelkha at redhat.com <lzelkha at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsDynamic.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDynamicDAODbFacadeImpl.java
2 files changed, 4 insertions(+), 5 deletions(-)

Approvals:
  Omer Frenkel: Looks good to me, but someone else must approve
  Liran Zelkha: Verified
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I30711bc02adcc6ebbcac95d609da0fbc54f49cb2
Gerrit-PatchSet: 12
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liran Zelkha <lzelkha at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Liran Zelkha <lzelkha at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list