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 <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Greg Padgett <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Ravi Nori <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
