Yaniv Bronhaim has posted comments on this change. Change subject: asyncore: Move generic reactor functionality ......................................................................
Patch Set 13: (1 comment) https://gerrit.ovirt.org/#/c/37055/13/lib/yajsonrpc/betterAsyncore.py File lib/yajsonrpc/betterAsyncore.py: Line 134: Line 135: Line 136: class Reactor(object): Line 137: def __init__(self): Line 138: self._map = {} > Sure I will add. Do you mean '_map'? you need to read python's asyncore doc to understand also how the dispatcher works and time poll loop so im not sure that pioter will be able to provide really meaningful and helpful explanation only for the socket map . lets try, but i don't want to delay this merge so long so lets add separate patch for docing also other areas Line 139: self._is_running = False Line 140: self._wakeupEvent = AsyncoreEvent(self._map) Line 141: Line 142: def add_dispatcher(self, disp): -- To view, visit https://gerrit.ovirt.org/37055 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I59a14b18f4c08d873763e1d3bd9d42b99de05fb1 Gerrit-PatchSet: 13 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <smizr...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Dima Kuznetsov <dkuzn...@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: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches