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

begin_nested() contextmanager error #201

Closed
hudecof opened this issue Nov 11, 2016 · 3 comments
Closed

begin_nested() contextmanager error #201

hudecof opened this issue Nov 11, 2016 · 3 comments

Comments

@hudecof
Copy link
Contributor

hudecof commented Nov 11, 2016

Hi,

when I try to use code

 async with conn.begin_nested() as trans:

Result is

17:19:10 RS.1   | -> async with conn.begin_nested() as trans:
17:19:14 RS.1   | (Pdb) AttributeError: __aexit__

Without context manager it's working as expected.

@asvetlov
Copy link
Member

Thanks for report.
.begin_nested() doesn't support async context manager.
Would you provide a patch?
Look like it is a simple task.

@hudecof
Copy link
Contributor Author

hudecof commented Nov 15, 2016

As I read the code you§re rigt, there is no support for CM for .begin_nested().
I can try to provide a patch but can't promise it will be soon.

@asvetlov
Copy link
Member

Closed by #208

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