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@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@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>