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(a)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(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>