Arik Hadas has posted comments on this change. Change subject: engine : Introduction of CommandExecutor and AsyncCommandCallBack ......................................................................
Patch Set 3: (1 comment) http://gerrit.ovirt.org/#/c/28154/3/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandExecutor.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandExecutor.java: Line 49: final CommandCallBack callBack) { Line 50: VdcReturnValueBase result = getBackendCommandObjectsHandler().runAction(command, null); Line 51: if (callBack != null) { Line 52: if (result.getSucceeded()) { Line 53: command.setCommandStatus(CommandStatus.SUCCEEDED); > The status Active is used when the command is executing, when the command h I see what you mean, but I think the terminology is confusing - as I see it, "when the command has completed successfully the status should be Succeeded" should be "when the execute phase of the command command has completed successfully the status should be SYNC_PHASE_Succeeded". IMO, command is completed successfully when all its tasks are finished successfully and its end-action phase is ended Line 54: } Line 55: callBack.setExecutionResult(result); Line 56: } Line 57: } -- To view, visit http://gerrit.ovirt.org/28154 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I70922b5b836971eca76dee26ba67b7023d546fe6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Greg Padgett <gpadg...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@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
