From Arik Hadas <ahadas@redhat.com>:

Arik Hadas has submitted this change and it was merged. Change subject: core: do not instantiate vm-manager when not needed ...................................................................... core: do not instantiate vm-manager when not needed Now that the instantiation of VmManager involves a query for the static data of a VM from the database, we should be more careful not to make redundant instantiations. This patch adds a machanism to retrieve the manager object of a specific VM only if it is already instantiated. This mechanism is used by methods that query content that is typically contained in VmManager, but when no such object exists, should use default values. Change-Id: I18e5ed3da7f1d5ee1b2a987f08eec4ba8965919a Bug-Url: https://bugzilla.redhat.com/1392209 Signed-off-by: Arik Hadas <ahadas@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java 2 files changed, 18 insertions(+), 4 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/65854 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I18e5ed3da7f1d5ee1b2a987f08eec4ba8965919a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>