Change in ovirt-engine[master]: core: minor optimization in vm-analyzer
ahadas at redhat.com
ahadas at redhat.com
Wed Jul 27 07:32:35 UTC 2016
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 at 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 at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
More information about the Engine-commits
mailing list