Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

misc(events): Add new optimized index #2345

Merged
merged 2 commits into from
Jul 30, 2024
Merged

misc(events): Add new optimized index #2345

merged 2 commits into from
Jul 30, 2024

Conversation

jdenquin
Copy link
Contributor

Context

  • We have pretty bad performances on the events table when we want to retreive events for aggregations and before computing it.

Description

  • Add the missing optimized index on external_subscription_id, code and timestamp and including organization_id and properties

@jdenquin jdenquin self-assigned this Jul 29, 2024
rsempe
rsempe reviewed Jul 29, 2024
View reviewed changes
@jdenquin jdenquin merged commit fb0ca06 into main Jul 30, 2024
6 checks passed
@jdenquin jdenquin deleted the events-new-index branch July 30, 2024 10:18
abdussamadbello pushed a commit to abdussamadbello/lago-api that referenced this pull request Aug 8, 2024
@jdenquin @abdussamadbello
misc(events): Add new optimized index (getlago#2345)
cce06f7 
## Context

- We have pretty bad performances on the `events` table when we want to
retreive events for aggregations and before computing it.

## Description

- Add the missing optimized index on `external_subscription_id`, `code`
and `timestamp` and including `organization_id` and `properties`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ancorcruz ancorcruz ancorcruz approved these changes

@nudded nudded nudded approved these changes

@rsempe rsempe rsempe approved these changes

@brunomiguelpinto brunomiguelpinto brunomiguelpinto approved these changes

@annvelents annvelents annvelents approved these changes

@ivannovosad ivannovosad Awaiting requested review from ivannovosad

@vincent-pochet vincent-pochet Awaiting requested review from vincent-pochet

@julienbourdeau julienbourdeau Awaiting requested review from julienbourdeau

@lovrocolic lovrocolic Awaiting requested review from lovrocolic

Assignees

@jdenquin jdenquin

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@jdenquin @ancorcruz @nudded @rsempe @brunomiguelpinto @annvelents