Allon Mureinik has submitted this change and it was merged. Change subject: core: VmValidator#vmNotHavingPassthroughVnics linq ...................................................................... core: VmValidator#vmNotHavingPassthroughVnics linq Replaced the usage of LinqUtils with a single fluent call to Java 8's streaming API. This patch both simplifies the code and offers a slight performance boost since the list of interfaces is only iterated once, instead of twice in the original code (once for filtering the passthrough nics and once for extracting their Names with ReplacementUtils). Change-Id: Ibb009e2c2e604a740712987fd35763c578f2b36c Signed-off-by: Allon Mureinik <amureini@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/VmValidator.java 1 file changed, 7 insertions(+), 9 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Verified Arik Hadas: Looks good to me, approved Yevgeny Zaspitsky: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/48891 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibb009e2c2e604a740712987fd35763c578f2b36c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>