Change in ovirt-engine[master]: core: minor cleanup in VmJobsMonitoring

ahadas at redhat.com ahadas at redhat.com
Thu Apr 21 09:38:04 UTC 2016


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



More information about the Engine-commits mailing list