Arik Hadas has submitted this change and it was merged. Change subject: core: minor optimization in vm-analyzer ...................................................................... core: minor optimization in vm-analyzer First check whether we get balloon information from VDSM and only then check if balloon is enabled on the host. This reason for this is that checking the host is a relatively costly operation in term of CPU time (specifically because of the cloning of VDS object). Not sure that the second check is needed anymore, but I prefer not to remove it as part of this patch. Change-Id: Ide71a7a5edcb44f05ca1b58e84d4ce3481fa5806 Signed-off-by: Arik Hadas <ahadas@redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61365 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide71a7a5edcb44f05ca1b58e84d4ce3481fa5806 Gerrit-PatchSet: 6 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>