-
Notifications
You must be signed in to change notification settings - Fork 296
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support asyncio redis #185
Conversation
@long2ice thanks for the pull request! There seems to be a minor bug related to importlib changes in Python 3.8. See: https://stackoverflow.com/questions/59216175/importerror-cannot-import-name-metadata-from-importlib |
Fixed |
It looks like the bug is still there:
|
Sorry, fixed again |
Thanks @long2ice . The issue is fixed. Would you mind add some documentation for this new feature? Currently the documentation only covers examples using MongoDB's Async IO API. It would be great to have a Redis version as well. This will increase the visibility of this new feature. The current documentation on Async IO: http://ekzhu.com/datasketch/lsh.html#asynchronous-minhash-lsh-at-scale The documentation source for this page: https://github.com/ekzhu/datasketch/blob/master/docs/lsh.rst |
OK, I will do it |
Done |
Thank you! |
No description provided.