Skip to content

Postgres: Index the parent_conv column in the conversation table - #5205

Merged
akshaymankar merged 4 commits into
developfrom
parent-conv-index
Apr 29, 2026
Merged

Postgres: Index the parent_conv column in the conversation table#5205
akshaymankar merged 4 commits into
developfrom
parent-conv-index

Conversation

@akshaymankar

Copy link
Copy Markdown
Member

This will reduce the time it takes to delete conversations.

https://wearezeta.atlassian.net/browse/WPB-22832

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

This will reduce the time it takes to delete conversations.
@akshaymankar
akshaymankar requested review from a team as code owners April 28, 2026 07:38
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Apr 28, 2026
Comment thread libs/wire-subsystems/src/Wire/PostgresMigrations.hs Outdated
akshaymankar and others added 2 commits April 28, 2026 16:30
Co-authored-by: Sven Tennie <sven.tennie@wire.com>

@supersven supersven left a comment

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.

The comment about Haddock is not very important. Other than that: LGTM 👍

markbrockhoff added a commit to wireapp/wire-webapp that referenced this pull request Apr 28, 2026
…420]

This test is currently broken due to the dev backend taking more than 10s to delete the conversation. (See: wireapp/wire-server#5205)
Until this issue has been resolved it will be skipped to unblock CI.
@akshaymankar
akshaymankar merged commit 57d5b15 into develop Apr 29, 2026
10 checks passed
@akshaymankar
akshaymankar deleted the parent-conv-index branch April 29, 2026 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants