Change in ovirt-engine[master]: core: simplify the logic within vm-jobs monitoring
Arik Hadas has submitted this change and it was merged.
Change subject: core: simplify the logic within vm-jobs monitoring
......................................................................
core: simplify the logic within vm-jobs monitoring
Instead of filtering the existing jobs from the reported jobs and
then iterating the reported jobs to find the jobs to update and
afterwards iterating the existing jobs to find the jobs to remove,
we now iterating only the existing jobs and find the jobs to update
and the jobs to remove in one cycle.
Change-Id: I71d3a6be42193a717a3c68b7c4dbe9966e1e080d
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmJobsMonitoring.java
1 file changed, 18 insertions(+), 31 deletions(-)
Approvals:
Shahar Havivi: Looks good to me, approved
Jenkins CI: Passed CI tests
Arik Hadas: Verified
--
To view, visit https://gerrit.ovirt.org/54314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I71d3a6be42193a717a3c68b7c4dbe9966e1e080d
Gerrit-PatchSet: 7
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: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>