Laszlo Hornyak has posted comments on this change. Change subject: core: vdsSelector: remove method used once (10/15) ......................................................................
Patch Set 1: (1 inline comment) .................................................... File backend/manager/modules/sla/src/main/java/org/ovirt/engine/core/sla/VdsSelector.java Line 101 Line 102 Line 103 Line 104 Line 105 I like the idea, but is this getDestinationVdsId is checked for null result wherever it is called? Are you sure it is safe to remove the null-check? -- To view, visit http://gerrit.ovirt.org/13212 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iecedcd6b9b754dbd853304d58083a556dab51cfa Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Laszlo Hornyak <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
