Skip to content

Document that the SQLite version included in Ubuntu LTS, aside from ESM-only versions, is included in our support policy.#19823

Merged
reivilibre merged 2 commits into
developfrom
rei/ubuntu_lts_sqlite
Jun 5, 2026
Merged

Document that the SQLite version included in Ubuntu LTS, aside from ESM-only versions, is included in our support policy.#19823
reivilibre merged 2 commits into
developfrom
rei/ubuntu_lts_sqlite

Conversation

@reivilibre

Copy link
Copy Markdown
Contributor

This was discussed months ago (private) and I never got around to writing it up here.

Signed-off-by: Olivier 'reivilibre <oliverw@matrix.org>
@reivilibre reivilibre marked this pull request as ready for review June 4, 2026 15:55
@reivilibre reivilibre requested a review from a team as a code owner June 4, 2026 15:55
Comment thread changelog.d/19823.doc
@@ -0,0 +1 @@
Document that the SQLite version included in Ubuntu LTS, aside from ESM-only versions, is included in our support policy. No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good to explain the context here for where this is spawning from.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not right sure how you mean; the context was whether I can use exciting new stuff in sliding sync, but it doesn't really feel worthwhile putting that in the changelog.
IMO the support policy is standalone and would be valid no matter what I was working on at the time to query.

But maybe you had something else in mind?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Explain it in the issue description / in this thread

IMO the support policy is standalone

Yes, but it's good to understand why we're making this change without having to look at the internal document to figure out the conversation which doesn't even encompass the entire context. For example, I just cross-linked some more conversation in the doc. And even that isn't fully linked to the relevant issues/PRs (#19452 and #19487)

Given you were the original author, I think you could give a great summary.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep that makes sense, so the motivation for wanting to not do this was that for support for SQLite's JSON operators, which is currently not present in the SQLite version found in Ubuntu's oldest supported LTS.

The JSON operators were used in some of the sticky events work (thank you for reminding me I opened #19452).

Our ruling was that we should support Ubuntu oldest LTS equally to Debian oldstable, so support the oldest of the two versions from those.

That makes some kind of sense as it would be difficult to do otherwise without dropping support for that version of Ubuntu altogether, given if we kept publishing packages intended for use with Postgres, there's a risk that an innocent sysadmin would update their SQLite deployment without realising that it is no longer supported.

@reivilibre reivilibre merged commit 7f45a1c into develop Jun 5, 2026
38 checks passed
@reivilibre reivilibre deleted the rei/ubuntu_lts_sqlite branch June 5, 2026 15:36
FrenchGithubUser pushed a commit to famedly/synapse-upstreaming that referenced this pull request Jun 12, 2026
…SM-only versions, is included in our support policy. (element-hq#19823)

The reason for querying this support was wanting support for SQLite's JSON operators, which are currently not present in the SQLite version found in Ubuntu's oldest supported LTS.

The JSON operators were used in some of the sticky events work (related: element-hq#19452).

Our ruling was that we should support Ubuntu oldest LTS equally to Debian oldstable, so support the oldest of the two versions from those.

That makes some kind of sense as it would be difficult to do otherwise without dropping support for that version of Ubuntu altogether, given if we kept publishing packages intended for use with Postgres, there's a risk that an innocent sysadmin would update their SQLite deployment without realising that it is no longer supported.

This was [discussed months ago
(private)](https://docs.google.com/document/d/12RZKPk3a4__JUSH9wYHODo9rRyKzsHg6BSCAcmqmbOU/edit?tab=t.0#bookmark=id.fcdvoc88dy5s)
and at [private](https://docs.google.com/document/d/12RZKPk3a4__JUSH9wYHODo9rRyKzsHg6BSCAcmqmbOU/edit?tab=t.0#bookmark=id.u48ivjge4qpt).

---------

Signed-off-by: Olivier 'reivilibre <oliverw@matrix.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants