PyZMQ 17.0.0 has deprecated and removed zmq.asyncio.install() call with advice to use asyncio native run-loop instead of zmq specific.
This caused exception when running the contrib/zmq/zmq_sub*.py examples.
This commit simply follows the advice and fixes mentioned examples.
PyZMQ 17.0.0 has deprecated and removed zmq.asyncio.install() call
with advice to use asyncio native run-loop instead of zmq specific.
This caused exception when running the contrib/zmq/zmq_sub*.py examples.
This commit simply follows the advice.
@randolf Changelog for PyZMQ 17.0.0 describes this change in poor wording:
Allow pyzmq asyncio/tornado integration to run without installing zmq_poll() implementation. The following methods and classes are deprecated and no longer required:
0zmq.eventloop.ioloop.install() 1zmq.eventloop.ioloop.IOLoop 2zmq.asyncio.install() 3zmq.asyncio.ZMQEventLoop
However, in the code of PyZMQ 17 the call to zmq.asyncio.install() is not only deprecated but also returns None:
0def _deprecated():
1 if _deprecated.called:
2 return
3 _deprecated.called = True
4 import warnings
5 warnings.warn("ZMQEventLoop and zmq.asyncio.install are deprecated in pyzmq 17. Special eventloop integration is no longer needed.", DeprecationWarning, stacklevel=3)
6_deprecated.called = False
7
8
9def install():
10 """DEPRECATED: No longer needed in pyzmq 17"""
11 _deprecated()
Even the documentation of PyZMQ no longer mentions zmq.asyncio.install() method.
This makes the example unusable in most recent version of PyZMQ.
I can add the condition for newest PyZMQ (and later) to choose right implementation of run-loop, but for the sake of brevity of example I think it’s not necessary.
