Change in ovirt-engine[master]: core: simplify the logic within vm-jobs monitoring

[ahadas at redhat.com]

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