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(a)redhat.com <lzelkha(a)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(a)redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server