Arik Hadas has submitted this change and it was merged. Change subject: core: retrieve vm-jobs only if needed ...................................................................... core: retrieve vm-jobs only if needed Previously, we queried vm-jobs from the database every time that VMs-monitoring was executed - even when no VM reported vm-jobs. Now, the vm-jobs are queried only if a VM reports VM-jobs. The query is now done by VmAnalyzer instead of VmsMonitoring. The drawback of this approach is that when we poll host with 100 running VMs, we will query the database 100 times in worse case (when all VMs report VM-jobs) instead of only once. However, the common case is that no VM or only a few report VM-jobs and thus this approach is better for the common case. For the extreme case a mechanism that caches the jobs instead of querying the database everytime would be the best solution. Change-Id: I2b9ee3c1bb0c6d7316a1a4f73488a550a1f994e5 Signed-off-by: Arik Hadas <ahadas(a)redhat.com&gt; --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmAnalyzer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java 2 files changed, 2 insertions(+), 14 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/52696 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b9ee3c1bb0c6d7316a1a4f73488a550a1f994e5 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com&gt; Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com&gt; Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com&gt; Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;