Shireesh Anjal has posted comments on this change. Change subject: engine,webadmin: RunMultipleAction - execution and error handling ......................................................................
Patch Set 1: The use case here was that user may select one or more rows from the table and click on an action button. In case user selects only one row, there is no reason to run asynchronously. In case of multiple though, it makes sense to trigger them asynchronously. It would have been nice to achieve this with a single call to the backend, instead of an if-then condition on every such screen to decide whether we should invoke RunAction or RunMultipleAction. However if the basic intention of RunMultipleAction is to run actions asynchronously, even in case of a single action, this patch is wrong and should be abandoned. -- To view, visit http://gerrit.ovirt.org/4995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I94116f4eb618f172ece1c90f14db5b139b7f1c4b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayi...@redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Shireesh Anjal <san...@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
