Arik Hadas has submitted this change and it was merged. Change subject: core: refactoring in SchedulingManager ......................................................................
core: refactoring in SchedulingManager SchedulingManager#schedule used have too many responsibilities - input validation, best host selection, update pending resources, and cluster locks handling. This patch extract the best host selection to separate method to remove one of the responsibilities mentioned above and simplify the code. In addition, other minor refactoring was made in this class. Change-Id: I57b8e4751b4da329658f4b8e1ae2366b1147a137 Signed-off-by: Arik Hadas <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java 1 file changed, 53 insertions(+), 47 deletions(-) Approvals: Arik Hadas: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24372 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I57b8e4751b4da329658f4b8e1ae2366b1147a137 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Doron Fediuck <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Martin Sivák <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
