Skip to content

Comments

Replace IN queries for convs with async#2397

Merged
pcapriotti merged 1 commit intodevelopfrom
pcapriotti/remove-in-queries
May 17, 2022
Merged

Replace IN queries for convs with async#2397
pcapriotti merged 1 commit intodevelopfrom
pcapriotti/remove-in-queries

Conversation

@pcapriotti
Copy link
Contributor

We were previously using IN queries when fetching multiple conversations. This can cause performance issues, as cassandra needs to contact all nodes. See [docs/legacy/developer/cassandra-interaction.md].

Checklist

  • The PR Title explains the impact of the change.
  • The PR description provides context as to why the change should occur and what the code contributes to that effect. This could also be a link to a JIRA ticket or a Github issue, if there is one.
  • changelog.d contains the following bits of information (details):
    • A file with the changelog entry in one or more suitable sub-sections. The sub-sections are marked by directories inside changelog.d.

We were previously using IN queries when fetching multiple
conversations. This can cause performance issues, as cassandra needs to
contact all nodes. See [docs/legacy/developer/cassandra-interaction.md].
@pcapriotti pcapriotti temporarily deployed to cachix May 16, 2022 12:19 Inactive
@pcapriotti pcapriotti merged commit ef2c5b3 into develop May 17, 2022
@pcapriotti pcapriotti deleted the pcapriotti/remove-in-queries branch May 17, 2022 08:04
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.

2 participants