Conversation
| if isinstance(val, RqlQuery): | ||
| return val | ||
| elif isinstance(val, collections.Callable): | ||
| elif isinstance(val, collections.abc.Callable): |
There was a problem hiding this comment.
change also needed for collections.Mapping
gabor-boros
left a comment
gabor-boros
left a comment
There was a problem hiding this comment.
This PR actually not updates the code to be python 3.10 compatible. Although the net_asyncio is "fixed", I'm not sure that the pinned dependencies, like tornado, supports python 3.10.
| self._parent.host, | ||
| self._parent.port, | ||
| loop=self._io_loop, | ||
| #loop=self._io_loop, droped at py3.10 |
There was a problem hiding this comment.
This makes the driver backward incompatible with older python versions.
There was a problem hiding this comment.
It does? How so? The parameter was optional and otherwise it uses the active loop.
|
Going to close, because, first, this only drops one of the loop parameters (there are a couple more that need to be dropped) and second, the change to And PR #296 removes all the loop params that need it. Also, the |
4 participants
To use with python 3.10 some little changes are necesary.
Callable uses collections.abc
Drop loop arg