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