Skip to content

feat(replays): Add segment_names column#8098

Draft
mjq wants to merge 2 commits into
masterfrom
mjq/replays-segment-names
Draft

feat(replays): Add segment_names column#8098
mjq wants to merge 2 commits into
masterfrom
mjq/replays-segment-names

Conversation

@mjq

@mjq mjq commented Jun 24, 2026

Copy link
Copy Markdown
Member

Not yet ready for review.

Fixes REPLAY-934.

@linear-code

linear-code Bot commented Jun 24, 2026

Copy link
Copy Markdown

REPLAY-934

@github-actions

Copy link
Copy Markdown

This PR has a migration; here is the generated SQL for ./snuba/migrations/groups.py ()

-- start migrations

-- forward migration replays : 0025_add_segment_names_column
Local op: ALTER TABLE replays_local ON CLUSTER 'cluster_one_sh' ADD COLUMN IF NOT EXISTS segment_names Array(String) AFTER urls;
Distributed op: ALTER TABLE replays_dist ON CLUSTER 'cluster_one_sh' ADD COLUMN IF NOT EXISTS segment_names Array(String) AFTER urls;
-- end forward migration replays : 0025_add_segment_names_column




-- backward migration replays : 0025_add_segment_names_column
Distributed op: ALTER TABLE replays_dist ON CLUSTER 'cluster_one_sh' DROP COLUMN IF EXISTS segment_names;
Local op: ALTER TABLE replays_local ON CLUSTER 'cluster_one_sh' DROP COLUMN IF EXISTS segment_names;
-- end backward migration replays : 0025_add_segment_names_column

The segment_names column needs an aggregate function since the replays
query uses GROUP BY replay_id.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant