Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Drop unused tables & unneeded access token ID for events. - #16268

Merged
clokep merged 2 commits into
developfrom
clokep/txn-clean-up
Oct 6, 2023
Merged

clokep merged 2 commits into
developfrom
clokep/txn-clean-up

Conversation

@clokep

@clokep clokep commented Sep 6, 2023

Copy link
Copy Markdown
Member

Fixes #16042 and fixes #15786.

Comment thread synapse/storage/databases/main/purge_events.py Outdated
@clokep

clokep commented Sep 12, 2023

Copy link
Copy Markdown
Member Author

#16273 will be in v1.93.0, so we'll want to wait a couple more versions until we can land this.

@clokep
clokep force-pushed the clokep/txn-clean-up branch from 23e49a3 to 0624832 Compare October 5, 2023 17:48
@clokep
clokep force-pushed the clokep/txn-clean-up branch from 0624832 to d40cb72 Compare October 5, 2023 17:49
@clokep

clokep commented Oct 5, 2023

Copy link
Copy Markdown
Member Author

develop is now v1.95.0's base so I think we can merge this.

@clokep
clokep marked this pull request as ready for review October 5, 2023 17:50
@clokep
clokep requested a review from a team as a code owner October 5, 2023 17:50
@clokep
clokep merged commit cabd577 into develop Oct 6, 2023
@clokep
clokep deleted the clokep/txn-clean-up branch October 6, 2023 12:29
clokep added a commit that referenced this pull request Oct 11, 2023
clokep added a commit that referenced this pull request Oct 12, 2023
…6268)" (#16465)

This reverts commit cabd577.

There are additional usages of these tables
which need to be removed first.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clean-up event_txn_id table Gracefully drop MSC2716 related tables (incrementally import history)

2 participants