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.

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

chore(backend): remove eventIds array from crash + anr groups #1145

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

anupcowkur
Copy link
Contributor

Description

Crash and ANR groups have a list of event ids associated with them which are stored as an array in the 'event_ids' column in Postgres. Postgres fields have a limit of 1GB and this list can get quite large in high scale environments.

To improve this situation, this commit:

  1. Removes the event_ids column from postgres
  2. Fetches event ids of crash + anr groups from clickhouse based on the event's fingeprint
  3. Updates the ingest algorithm to no longer append event_ids to crash & anr groups and only update updated_at and first_seen_event timestamps
  4. Updates the cleanup service to no longer delete event_ids from postgres on stale event cleanup

Related issue

Closes #1027

@anupcowkur anupcowkur requested a review from detj August 28, 2024 11:27
Copy link

vercel bot commented Aug 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
measure-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 29, 2024 10:52am

closes #1027

Crash and ANR groups have a list of event ids associated with them
which are stored as an array in the 'event_ids' column in postgres.
Postgres fields have a limit of 1GB and this  list can get quite
large in high scale environments.

To improve this situation, this commit:
1. Removes the event_ids column from postgres
2. Fetches event ids of crash + anr groups from clickhouse based
on the event's fingeprint
3. Updates the ingest algorithm to no longer append event_ids to
crash & anr groups and only update updated_at and first_seen_event
timestamps
4. Updates the cleanup service to no longer delete event_ids
from postgres on stale event cleanup
@anupcowkur anupcowkur force-pushed the remove-event-ids-from-groups branch from 60c8dab to 6c8e62a Compare August 29, 2024 10:51
@anupcowkur anupcowkur merged commit da73bb7 into main Aug 29, 2024
11 checks passed
@anupcowkur anupcowkur deleted the remove-event-ids-from-groups branch August 29, 2024 10:54
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.

remove event IDs array in anr and crash groups from postgres
2 participants