Dan Kenigsberg has submitted this change and it was merged. Change subject: stomp: use the atomic dict.setdefault() instead for check and set ......................................................................
stomp: use the atomic dict.setdefault() instead for check and set This is faster and simpler. Change-Id: I28e4925a78328c4ac6cd0b9fc8659d6a3ee27a8f Signed-off-by: Saggi Mizrahi <smizr...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/35937 Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com> Reviewed-by: Dan Kenigsberg <dan...@redhat.com> Reviewed-by: Piotr Kliczewski <piotr.kliczew...@gmail.com> Reviewed-by: Francesco Romani <from...@redhat.com> Tested-by: Piotr Kliczewski <piotr.kliczew...@gmail.com> --- M lib/yajsonrpc/stomp.py 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Piotr Kliczewski: Verified; Looks good to me, but someone else must approve Yaniv Bronhaim: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Francesco Romani: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/35937 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28e4925a78328c4ac6cd0b9fc8659d6a3ee27a8f Gerrit-PatchSet: 12 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <smizr...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches