Skip to content

Conversation

carlobeltrame
Copy link
Member

@carlobeltrame carlobeltrame commented Oct 4, 2025

Follow-up for #8004 after deploying that feature to staging and performing a load test using Grafana K6.

Querying by isPrototype OR isShared is slow (with our productive amount of data in the order of magnitude 100 times slower), because databases don't like OR conditions too much. We work around this by adding a combined isPublic field which is enforced to always contain isShared || isPrototype and have the DB queries filter by that.

@carlobeltrame carlobeltrame requested a review from a team October 4, 2025 10:39
@carlobeltrame carlobeltrame added the deploy! Creates a feature branch deployment for this PR label Oct 4, 2025
Follow-up for ecamp#8004

Querying by isPrototype OR isShared is slow, because databases don't
like OR conditions too much. We work around this by adding a combined
isPublic field which is enforced to always contain isShared || isPrototype
and have the DB queries filter by that.
Copy link

github-actions bot commented Oct 4, 2025

Feature branch deployment currently inactive.

If the PR is still open, you can add the deploy! label to this PR to trigger a feature branch deployment.

@carlobeltrame carlobeltrame force-pushed the shared-camps-better-performance branch from 25d9ca4 to a54618b Compare October 4, 2025 10:41
@pmattmann pmattmann removed the deploy! Creates a feature branch deployment for this PR label Oct 4, 2025
@pmattmann pmattmann added the deploy! Creates a feature branch deployment for this PR label Oct 4, 2025
@carlobeltrame carlobeltrame added this pull request to the merge queue Oct 4, 2025
Merged via the queue into ecamp:devel with commit 46c65c5 Oct 4, 2025
41 checks passed
@carlobeltrame carlobeltrame deleted the shared-camps-better-performance branch October 4, 2025 11:14
This was referenced Oct 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy! Creates a feature branch deployment for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants