Skip to content

chore(ml/replay): nuke session replay features tables#59044

Merged
nicowaltz merged 5 commits into
masterfrom
nico/nuke-features-table
May 20, 2026
Merged

chore(ml/replay): nuke session replay features tables#59044
nicowaltz merged 5 commits into
masterfrom
nico/nuke-features-table

Conversation

@nicowaltz
Copy link
Copy Markdown
Contributor

Problem

Migration 252 broke the session_replay_features table and left orphaned views.

Changes

Nukes all session_replay_features tables in order to clean the slate for recreating them. This is OK because the session feature recorder is currently DISABLED and will remain so until the tables are recreated.

@nicowaltz nicowaltz requested a review from a team as a code owner May 19, 2026 15:50
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 19, 2026

🎭 Playwright didn't run on this PR — your changes touch code that could affect E2E behavior, but Playwright is opt-in via label now to keep CI cost down.

Add the run-playwright label if you want an E2E sweep before merging — CI will pick it up automatically.

Most PRs don't need this. Real regressions still get caught on master and fix-forward.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 19, 2026

Reviews (1): Last reviewed commit: "update max_migrations" | Re-trigger Greptile

# 4. Read-side Distributed table.
run_sql_with_exceptions(
"DROP TABLE IF EXISTS session_replay_features",
node_roles=[NodeRole.AUX],
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if running one query on multiple node_roles, just use

node_roles = [NodeRole.AUX, NodeRole.DATA]

nicowaltz added 2 commits May 20, 2026 16:32
…table

# Conflicts:
#	posthog/clickhouse/migrations/max_migration.txt
@nicowaltz nicowaltz enabled auto-merge (squash) May 20, 2026 15:35
@nicowaltz nicowaltz merged commit ab36548 into master May 20, 2026
218 checks passed
@nicowaltz nicowaltz deleted the nico/nuke-features-table branch May 20, 2026 16:01
@deployment-status-posthog
Copy link
Copy Markdown

deployment-status-posthog Bot commented May 20, 2026

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-05-20 17:34 UTC Run
prod-us ✅ Deployed 2026-05-20 17:53 UTC Run
prod-eu ✅ Deployed 2026-05-20 17:52 UTC Run

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