Skip to content

Default isolation level None breaks Transaction.__repr__ #702

New issue

Have a question about this project? No Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “No 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? No Sign in to your account

Closed
BeatButton opened this issue Feb 15, 2021 · 0 comments · Fixed by #703
Closed

Default isolation level None breaks Transaction.__repr__ #702

BeatButton opened this issue Feb 15, 2021 · 0 comments · Fixed by #703

Comments

@BeatButton
Copy link
Contributor

  • asyncpg version: 0.22.0
  • PostgreSQL version: 13.2
  • Do you use a PostgreSQL SaaS? If so, which? Can you reproduce
    the issue with a local PostgreSQL install?
    : No, I do not.
  • Python version: 3.9.1
  • Platform: Ubuntu 20.10
  • Do you use pgbouncer?: No
  • Did you install asyncpg with pip?: Yes
  • If you built asyncpg locally, which version of Cython did you use?: N/A
  • Can the issue be reproduced under both asyncio and
    uvloop?
    : Yes

Since Transaction.__repl__ does ' '.join(attrs), and attrs contains self._isolation, it will fail with a TypeError with the new default isolation level of None.

Regression introduced in #622

@BeatButton BeatButton changed the title Default isolation level None breaks Transaction.__repl__ Default isolation level None breaks Transaction.__repr__ Feb 15, 2021
No Sign up for free to join this conversation on GitHub. Already have an account? No Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant