Liron Aravot has uploaded a new change for review. Change subject: core: ExportVm - remove unneeded VmInit load ......................................................................
core: ExportVm - remove unneeded VmInit load The load of the VmInit isn't relevant on the Execute part of the execution, becasue the vm data isn't used on that phase. When the vm data is used the VmInit is already being loaded, so the call is redundant. Change-Id: Ia365912c7faf36670f95a96ed5731db7de3fed71 Signed-off-by: [email protected] <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportVmCommand.java 1 file changed, 0 insertions(+), 3 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/99/38199/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportVmCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportVmCommand.java index 666bbce..82edd22 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportVmCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportVmCommand.java @@ -229,9 +229,6 @@ VmHandler.lockVm(getVm().getDynamicData(), getCompensationContext()); freeLock(); - // update vm init - VmHandler.updateVmInitFromDB(getVm().getStaticData(), true); - // Means that there are no asynchronous tasks to execute - so we can end the command // immediately after the execution of the previous steps if (!hasSnappableDisks() && snapshotsWithMemory.isEmpty()) { -- To view, visit https://gerrit.ovirt.org/38199 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ia365912c7faf36670f95a96ed5731db7de3fed71 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
