Yair Zaslavsky has posted comments on this change. Change subject: aaa: use runVdsCommand ......................................................................
Patch Set 14: (2 comments) http://gerrit.ovirt.org/#/c/29049/14/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java: Line 1939: * know why). Line 1940: */ Line 1941: protected VDSReturnValue runVdsCommand(VDSCommandType commandType, VDSParametersBase parameters) Line 1942: throws VdcBLLException { Line 1943: parameters.setCorrelationId(getCorrelationId()); > we still have this in thread local... why is it needed in this patch? true, i will remove. Line 1944: return getBackend().getResourceManager().RunVdsCommand(commandType, parameters); Line 1945: } Line 1946: Line 1947: /** http://gerrit.ovirt.org/#/c/29049/14/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSParametersBase.java File backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSParametersBase.java: Line 2: Line 3: public class VDSParametersBase { Line 4: Line 5: private boolean runAsync; Line 6: private String correlationId; > is this belong to this patch? yep, see my comment before. removing. Line 7: Line 8: public VDSParametersBase() { Line 9: runAsync = true; Line 10: } -- To view, visit http://gerrit.ovirt.org/29049 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Idcf48db5ea278a74b1bf87076d3b16af29bb416c Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Ravi Nori <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Sahina Bose <[email protected]> Gerrit-Reviewer: Tal Nisan <[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
