Skip to content

sqlite: Ignore ExecuteReturnedResults on PRAGMA calls - #3561

Merged
badboy merged 1 commit into
mainfrom
sqlite-ignore-executereturnedr/lrvnqrwrupun
Jul 31, 2026
Merged

sqlite: Ignore ExecuteReturnedResults on PRAGMA calls#3561
badboy merged 1 commit into
mainfrom
sqlite-ignore-executereturnedr/lrvnqrwrupun

Conversation

@badboy

@badboy badboy commented Jul 29, 2026

Copy link
Copy Markdown
Member

There's rare occasions where that error is returned for ... I don't know why. I couldn't get query_row_and_then to work on that same pragma either.

This code is copied from appservices:
https://github.com/mozilla/application-services/blob/407a2421c6f731b921894927cdf9a6bacaa3813d/components/support/sql/src/conn_ext.rs#L45-L63


fwiw hitting this error seems to be what's causing the iOS errors on #3538

There's rare occasions where that error is returned for ... I don't know
why. I couldn't get `query_row_and_then` to work on that same pragma either.

This code is copied from appservices:
https://github.com/mozilla/application-services/blob/407a2421c6f731b921894927cdf9a6bacaa3813d/components/support/sql/src/conn_ext.rs#L45-L63
@badboy
badboy requested a review from a team as a code owner July 29, 2026 09:43
@badboy
badboy requested review from jeddai and removed request for a team July 29, 2026 09:43
@badboy badboy added the sqlite Any changes to the new SQLite storage backend label Jul 29, 2026
@badboy
badboy enabled auto-merge (rebase) July 29, 2026 09:44
@badboy
badboy requested a review from chutten July 29, 2026 16:00
@badboy
badboy merged commit 7866dc1 into main Jul 31, 2026
33 of 34 checks passed
@badboy
badboy deleted the sqlite-ignore-executereturnedr/lrvnqrwrupun branch July 31, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sqlite Any changes to the new SQLite storage backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants