Daniel Erez has submitted this change and it was merged. Change subject: webadmin: MoveOrCopyDiskModel: postInitStorageDomains refactoring. ...................................................................... webadmin: MoveOrCopyDiskModel: postInitStorageDomains refactoring. postInitStorageDomains function was using multiple filtering on the same list of storage domains to create destination storage domain list for each disk model in this model. Going through this list again and again is redundant, thus this patch changes it to makes a single run on the domains list and builds the destination array in one iteration. Change-Id: I7538a341c3a7059b0a6ee369ca1a9b33648df456 Signed-off-by: Amit Aviram <aaviram(a)redhat.com&gt; --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Linq.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/MoveOrCopyDiskModel.java 2 files changed, 56 insertions(+), 65 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Amit Aviram: Verified -- To view, visit http://gerrit.ovirt.org/36555 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7538a341c3a7059b0a6ee369ca1a9b33648df456 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Amit Aviram <aaviram(a)redhat.com&gt; Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com&gt; Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com&gt; Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server