Skip to content
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

Synchronous API should be deprecated/removed. #61

Open
yglukhov opened this issue Nov 16, 2018 · 2 comments
Open

Synchronous API should be deprecated/removed. #61

yglukhov opened this issue Nov 16, 2018 · 2 comments

Comments

@yglukhov
Copy link
Contributor

It is distracting, useless, untested, and can lure some users into using it, and migrating from sync to async takes some time, so why not start with async in the first place. Oh, also it is a maintenance burden :)

@timotheecour
Copy link
Contributor

timotheecour commented Nov 16, 2018

just saw your post after submitting #63 ; for the maintenance (+ testing) burden, #63 could help

@timotheecour
Copy link
Contributor

timotheecour commented Nov 16, 2018

@yglukhov how would I translate this to async?

  for ai in mc.find(query).limit(2).items():
    echo ai

EDIT: see 3ada6cd for my approach to this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants