Skip to content

fix: reduce number of repeat_vars() calls#6358

Merged
link2xt merged 1 commit intomainfrom
link2xt/no-repeat-vars
Dec 22, 2024
Merged

fix: reduce number of repeat_vars() calls#6358
link2xt merged 1 commit intomainfrom
link2xt/no-repeat-vars

Conversation

@link2xt
Copy link
Copy Markdown
Collaborator

@link2xt link2xt commented Dec 22, 2024

SQL statements fail if the number of variables
exceeds SQLITE_LIMIT_VARIABLE_NUMBER.

Remaining repeat_vars() calls are difficult to replace
and use arrays passed from the UI,
e.g. forwarded message IDs or marked as seen IDs.

Closes #6355

@link2xt link2xt force-pushed the link2xt/no-repeat-vars branch 3 times, most recently from e841601 to 0550ef3 Compare December 22, 2024 19:25
@link2xt link2xt marked this pull request as ready for review December 22, 2024 19:25
@link2xt link2xt force-pushed the link2xt/no-repeat-vars branch from 0550ef3 to feecaa5 Compare December 22, 2024 19:58
SQL statements fail if the number of variables
exceeds `SQLITE_LIMIT_VARIABLE_NUMBER`.

Remaining repeat_vars() calls are difficult to replace
and use arrays passed from the UI,
e.g. forwarded message IDs or marked as seen IDs.
@link2xt link2xt force-pushed the link2xt/no-repeat-vars branch from feecaa5 to 78e0b19 Compare December 22, 2024 19:59
@link2xt link2xt enabled auto-merge (rebase) December 22, 2024 20:04
@link2xt link2xt merged commit 3d5e442 into main Dec 22, 2024
@link2xt link2xt deleted the link2xt/no-repeat-vars branch December 22, 2024 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SQL errors while trying to move / delete messages

2 participants