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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Amit Aviram <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
