Skip to content

bpo-44958: Only reset sqlite3 statements when needed #27844

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

Merged
pablogsal merged 16 commits into from
Sep 21, 2021
Merged

bpo-44958: Only reset sqlite3 statements when needed #27844

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
99adc05
bpo-44958: Only reset sqlite3 statements when needed
Aug 19, 2021
2f8cb23
We don't use the return value of pysqlite_statement_reset
Aug 20, 2021
e6e4973
If possible, avoid resetting unused statements
Aug 20, 2021
df769c3
SQLITE_STMTSTATUS_RUN is available from 3.20.0 and onwards
Aug 20, 2021
cbcd640
Use sqlite3_stmt pointer iso. deref'ing self when calling sqlite3_reset
Aug 20, 2021
fe0fd6f
Merge branch 'main' into sqlite-optimise-resets
Aug 22, 2021
0633095
Sync with 'main' bco. GH-27884
Aug 25, 2021
06e242c
pysqlite_statement_reset result() is never used => return void
Aug 25, 2021
747000c
Sync with `main` bco. GH-27942
Aug 25, 2021
4ad7e99
Make sure we reset statements both at cursor dealloc and when the
Sep 6, 2021
1bbb31f
Harden test_type_map_usage()
Sep 6, 2021
73e1ddb
Add regression tests for stmt exit from cursor
Sep 6, 2021
9684593
Merge branch 'main' into sqlite-optimise-resets
Sep 6, 2021
41410e0
Merge branch 'main' into sqlite-optimise-resets
Sep 14, 2021
ebebdb7
Merge branch 'main' into sqlite-optimise-resets
Sep 20, 2021
c84311e
Test that datamodifying select queries are marked readonly
Sep 20, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
If possible, avoid resetting unused statements
  • Loading branch information
Erlend E. Aasland committed Aug 20, 2021
commit e6e4973f08cc77fbdd6abf955d6e42104f6970f5
28 changes: 18 additions & 10 deletions Modules/_sqlite/statement.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -362,20 +362,28 @@ pysqlite_statement_bind_parameters(pysqlite_state *state,

int pysqlite_statement_reset(pysqlite_Statement* self)
{
int rc;
sqlite3_stmt *stmt = self->st;
if (stmt == NULL || self->in_use == 0) {
return SQLITE_OK;
}

rc = SQLITE_OK;
#if SQLITE_VERSION_NUMBER > 3020000
/* Check if the statement has been run (that is, sqlite3_step() has been
* called at least once). Third parameter is non-zero in order to reset the
* run count. */
if (sqlite3_stmt_status(stmt, SQLITE_STMTSTATUS_RUN, 1) == 0) {
return SQLITE_OK;
}
#endif

if (self->in_use && self->st) {
Py_BEGIN_ALLOW_THREADS
rc = sqlite3_reset(self->st);
Py_END_ALLOW_THREADS
int rc;
Py_BEGIN_ALLOW_THREADS
rc = sqlite3_reset(self->st);
Py_END_ALLOW_THREADS

if (rc == SQLITE_OK) {
self->in_use = 0;
}
if (rc == SQLITE_OK) {
self->in_use = 0;
}

return rc;
}

Expand Down