Liran Zelkha has uploaded a new change for review. Change subject: engine: Eliminate unneccessary VdsDynamic updates ......................................................................
engine: Eliminate unneccessary VdsDynamic updates Eliminiate unneccessary VdsDynamic updates in VdsUpdateRuntimeInfo. For short - no updates go to the DB if the object is the same as in the database. Change-Id: I152fb75da6eb5731ca41309d1704fe52ac1398e0 Signed-off-by: [email protected] <[email protected]> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java 1 file changed, 5 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/83/17483/1 diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java index 0e5b7df..4029107 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java @@ -397,7 +397,11 @@ * @param dynamicData */ public void UpdateDynamicData(VdsDynamic dynamicData) { - DbFacade.getInstance().getVdsDynamicDao().update(dynamicData); + if (dynamicData != vdsDynamic) { + DbFacade.getInstance().getVdsDynamicDao().update(dynamicData); + } else { + log.debug("Ignoring an unneccessary update on vdsDynamic " + vdsStatic.getName()); + } } /** -- To view, visit http://gerrit.ovirt.org/17483 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I152fb75da6eb5731ca41309d1704fe52ac1398e0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
