Arik Hadas has submitted this change and it was merged. Change subject: core: minor cleanup in VmJobsMonitoring ...................................................................... core: minor cleanup in VmJobsMonitoring 1. Break long chain of calls in getExistingJobsForVm to multiple lines. 2. Rename vmJobsToRemove and vmJobsToUpdate to jobsToRemove and jobsToUpdate as in this context it is clear that it is about VM-jobs. Change-Id: I182927c2b9a693564b9780caed9cf7912e9702a0 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, 10 insertions(+), 7 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/56246 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I182927c2b9a693564b9780caed9cf7912e9702a0 Gerrit-PatchSet: 3 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: Shahar Havivi <shavivi@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>