Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription ......................................................................
Patch Set 19: Code-Review-1
(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):
This is contract that we use for setting a message handler.
did you sign on that? at least call it from the constructor with none. don't you need to set this handler also to the subscription\sub (please change) ? it smalls bad Line 337: self._message_handler = handler Line 338: Line 339: def send(self, data): Line 340: headers = {