-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
Improve clarity of sqlite3 transaction handling docs #94017
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
Comments
Just to note, this confused me greatly at first, since I thought it was referring to an |
Sorry 'bout that! I was in a hurry :) |
Co-authored-by: Alex Waygood <[email protected]> Co-authored-by: CAM Gerlach <[email protected]>
…ythonGH-94320) Co-authored-by: Alex Waygood <[email protected]> Co-authored-by: CAM Gerlach <[email protected]> (cherry picked from commit 760b8cf) Co-authored-by: Erlend Egeberg Aasland <[email protected]>
…ythonGH-94320) Co-authored-by: Alex Waygood <[email protected]> Co-authored-by: CAM Gerlach <[email protected]> (cherry picked from commit 760b8cf) Co-authored-by: Erlend Egeberg Aasland <[email protected]>
) Co-authored-by: Alex Waygood <[email protected]> Co-authored-by: CAM Gerlach <[email protected]> (cherry picked from commit 760b8cf) Co-authored-by: Erlend Egeberg Aasland <[email protected]>
) Co-authored-by: Alex Waygood <[email protected]> Co-authored-by: CAM Gerlach <[email protected]> (cherry picked from commit 760b8cf) Co-authored-by: Erlend Egeberg Aasland <[email protected]>
Documentation
The sqlite3 module docs has a section called Controlling Transactions.
There are some improvement possibilities:
execute
orexecutemany
, but not byexecutescript
. Other DML statements do not receive such treatment.execute
,executemany
, andexecutescript
cursor methods to briefly mention their transaction control peculiarities.in_transaction
connection attribute. Such information can help developers debug transaction control problems.isolation_level
attribute. Currently, theisolation_level
property ofconnection
refer to the docs of theisolation_level
attribute, which again refer to the Controlling Transactions section. Theisolation_level
property should have a concise description (reference), and link to theisolation_level
attribute. Theisolation_level
attribute should provide a reference for what it does, and briefly mention expected behaviour. The Controlling Transactions section should provide more background (explanation) to what is happening, and it should briefly mention PEP 249, and the SQLite autocommit mode.The text was updated successfully, but these errors were encountered: