Ravi Nori has posted comments on this change. Change subject: engine : Introduction of CommandExecutor and AsyncCommandCallBack ......................................................................
Patch Set 3: (2 comments) 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); > shouldn't it be active? The status Active is used when the command is executing, when the command has completed successfully the status should be Succeeded. But CommandExecutor is not supposed to set the status, so I will be removing it. Line 54: } Line 55: callBack.setExecutionResult(result); Line 56: } Line 57: } http://gerrit.ovirt.org/#/c/28154/3/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/interfaces/CommandCallBack.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/interfaces/CommandCallBack.java: Line 2: Line 3: import org.ovirt.engine.core.common.action.VdcReturnValueBase; Line 4: Line 5: public interface CommandCallBack { Line 6: public abstract void setExecutionResult(VdcReturnValueBase result); > why abstract? Done -- 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
