Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription ......................................................................
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stompReactor.py File lib/yajsonrpc/stompReactor.py:
Line 332: self._destination = destination Line 333: self._client = client Line 334: self._message_handler = lambda arg: None Line 335: Line 336: def setMessageHandler(self, handler):
did you sign on that? at least call it from the constructor with none. don'
I understand that is not easy to follow which object is actually handling the messages will create docstring to clarify. Line 337: self._message_handler = handler Line 338: Line 339: def send(self, data): Line 340: headers = {