Oved Ourfali has submitted this change and it was merged. Change subject: core: remove redundant synchronization (CoCo cleanup) ......................................................................
core: remove redundant synchronization (CoCo cleanup) a method which is executed by our quartz in fixed delay can only be executed by one thread at each given time, because before executing it we remove it from the quartz pool and when it ends we put it back. so no need to synchronize anything for protecting from parallel executions of the same critical section Change-Id: I588de14e21f19ef59781b382b2450b8723635667 Signed-off-by: Arik Hadas <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandExecutor.java 1 file changed, 5 insertions(+), 10 deletions(-) Approvals: Ravi Nori: Looks good to me, but someone else must approve Arik Hadas: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29522 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I588de14e21f19ef59781b382b2450b8723635667 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Ravi Nori <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
