Change in ovirt-engine[master]: core: more efficient query for getting vms to monitor

shavivi at redhat.com shavivi at redhat.com
Sun Jun 5 10:02:07 UTC 2016


Shahar Havivi has submitted this change and it was merged.

Change subject: core: more efficient query for getting vms to monitor
......................................................................


core: more efficient query for getting vms to monitor

Once in every 5th iteration (currently every 15 secs) we query all the VMs that
run on a given host for the sake of monitoring. We used to query them using the
'vms' view. 'vms' view get all the data of the VMs from vm_static, vm_dynamic,
vm_statistics, cluster, storage_pool, quota, vds_static, vm_pool_map_view and
snapshots, while the data needed by the monitoring resides only in vm_static,
vm_dynamic and vm_statistics.

This patch introduces another query dedicated for monitoring that is much
more efficient than the previous one.

Change-Id: Iac9d32f7bc47f7e7af9075e52b658b00cc7f64c3
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDaoImpl.java
M packaging/dbscripts/create_views.sql
M packaging/dbscripts/vms_sp.sql
3 files changed, 74 insertions(+), 1 deletion(-)

Approvals:
  Eli Mesika: Looks good to me, but someone else must approve
  Shahar Havivi: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Arik Hadas: Verified



-- 
To view, visit https://gerrit.ovirt.org/57535
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iac9d32f7bc47f7e7af9075e52b658b00cc7f64c3
Gerrit-PatchSet: 17
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: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <mskrivan at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list