Skip to content

gh-84943: Add support for 'directonly' and 'innocuous' flags for user-defined functions - #149961

Open
jbosboom wants to merge 1 commit into
python:mainfrom
jbosboom:sqlite-innocuous-directonly
Open

gh-84943: Add support for 'directonly' and 'innocuous' flags for user-defined functions#149961
jbosboom wants to merge 1 commit into
python:mainfrom
jbosboom:sqlite-innocuous-directonly

Conversation

@jbosboom

@jbosboom jbosboom commented May 17, 2026

Copy link
Copy Markdown
Contributor

Based on #20398, hence Co-authored-by. The flags have also been added to create_window_function, which did not exist at the time. apply_deterministic_flag_if_supported was dropped because all stable Python versions require SQLite version 3.15.2 or later. The other two apply-flag-if-supported functions were modified to only raise when the flag is True. The tests were moved to their new location with updated style and some unnecessary SQL statements were dropped. The documentation now links to the SQLite documentation, which seems to be the prevailing style, rather than repeating it inline.

@read-the-docs-community

read-the-docs-community Bot commented May 17, 2026

Copy link
Copy Markdown

@jbosboom
jbosboom force-pushed the sqlite-innocuous-directonly branch from 725a8cc to 9700f31 Compare May 17, 2026 20:23
@jbosboom

Copy link
Copy Markdown
Contributor Author

The CI job failed due to a DNS problem. The other sanitizer jobs succeeded, so I think it just needs to be re-run.

@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 90 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Aug 19, 2026
…r user-defined functions

Co-authored-by: Erlend E. Aasland <erlend@python.org>
@jbosboom
jbosboom force-pushed the sqlite-innocuous-directonly branch from 9700f31 to 45b8215 Compare August 19, 2026 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review stale Stale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant