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
Improve clarity of sqlite3 transaction handling docs #94017
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 <Alex.Waygood@Gmail.com> Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM>
…ythonGH-94320) Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com> Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM> (cherry picked from commit 760b8cf) Co-authored-by: Erlend Egeberg Aasland <erlend.aasland@innova.no>
…ythonGH-94320) Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com> Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM> (cherry picked from commit 760b8cf) Co-authored-by: Erlend Egeberg Aasland <erlend.aasland@innova.no>
) Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com> Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM> (cherry picked from commit 760b8cf) Co-authored-by: Erlend Egeberg Aasland <erlend.aasland@innova.no>
) Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com> Co-authored-by: CAM Gerlach <CAM.Gerlach@Gerlach.CAM> (cherry picked from commit 760b8cf) Co-authored-by: Erlend Egeberg Aasland <erlend.aasland@innova.no>
Documentation
The sqlite3 module docs has a section called Controlling Transactions.
There are some improvement possibilities:
executeorexecutemany, but not byexecutescript. Other DML statements do not receive such treatment.execute,executemany, andexecutescriptcursor methods to briefly mention their transaction control peculiarities.in_transactionconnection attribute. Such information can help developers debug transaction control problems.isolation_levelattribute. Currently, theisolation_levelproperty ofconnectionrefer to the docs of theisolation_levelattribute, which again refer to the Controlling Transactions section. Theisolation_levelproperty should have a concise description (reference), and link to theisolation_levelattribute. Theisolation_levelattribute 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: