Alona Kaplan has posted comments on this change. Change subject: webadmin: Aggregate errors when importing networks ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/25838/2/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java File frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java: Line 627: callbacks[n - 1] = successCallback; Line 628: runMultipleActions(actionType, parameters, Line 629: new LinkedList<IFrontendActionAsyncCallback>(Arrays.asList(callbacks)), Line 630: state, Line 631: false); I think the default should be there is aggregation. Same as in runMultipleAction. Line 632: } Line 633: Line 634: /** Line 635: * A convenience method that calls {@link #runMultipleActions(VdcActionType, List, List, Object)} with -- To view, visit http://gerrit.ovirt.org/25838 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5230847a055a8a58f4596290e9ee5d2af2b09784 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lver...@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com> Gerrit-Reviewer: Lior Vernia <lver...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
