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.

Sign up for GitHub

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

Jump to bottom

Using SQLAlchemy with AUTOCOMMIT causes warnings from dbapi #494

Closed
olavloite opened this issue Nov 6, 2024 · 0 comments · Fixed by googleapis/python-spanner#1249
Closed

Using SQLAlchemy with AUTOCOMMIT causes warnings from dbapi #494

olavloite opened this issue Nov 6, 2024 · 0 comments · Fixed by googleapis/python-spanner#1249
Assignees
@olavloite
Labels
api: spanner Issues related to the googleapis/python-spanner-sqlalchemy API. priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@olavloite
Copy link
Contributor

Setting the 'isolation level' to AUTOCOMMIT causes the dbapi driver to emit a warning when SQLAlchemy calls COMMIT. This warning should be skipped.
@olavloite olavloite added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. priority: p2 Moderately-important priority. Fix may not be included in next release. labels Nov 6, 2024
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/python-spanner-sqlalchemy API. label Nov 6, 2024
@olavloite olavloite assigned olavloite and unassigned harshachinta Nov 7, 2024
olavloite added a commit to googleapis/python-spanner that referenced this issue Dec 3, 2024
@olavloite
feat: add connection variable for ignoring transaction warnings
da797a3 
Adds a connection variable for ignoring transaction warnings. Also adds
a **kwargs argument to the connect function. This will be used for
further connection variables in the future.

Fixes googleapis/python-spanner-sqlalchemy#494
@olavloite olavloite mentioned this issue Dec 3, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@olavloite olavloite

Labels
api: spanner Issues related to the googleapis/python-spanner-sqlalchemy API. priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add connection variable for ignoring transaction warnings googleapis/python-spanner
2 participants
@olavloite @harshachinta