Skip to content

fix(supabase): fix classification of schema changes with COMMENT ON PUBLICATION - #6440

Merged
darkgnotic merged 1 commit into
mainfrom
darkgnotic/fix-comment-on-publication-misclassification
Aug 29, 2026
Merged

fix(supabase): fix classification of schema changes with COMMENT ON PUBLICATION#6440
darkgnotic merged 1 commit into
mainfrom
darkgnotic/fix-comment-on-publication-misclassification

Conversation

@darkgnotic

Copy link
Copy Markdown
Contributor

Fix the classification of schema change type when processing a COMMENT ON PUBLICATION tag. Examining the previous schema change is incorrect as in Supabase the ALTER PUBLICATION will be missing. So always consider a schema change detected by COMMENT ON PUBLICATION as "unknown" which will correctly initiate a backfill for new tables/columns.

Context:

@darkgnotic
darkgnotic requested a review from tantaman August 29, 2026 01:14
@vercel

vercel Bot commented Aug 29, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
replicache-docs Ready Ready Preview Aug 29, 2026 1:14am
zbugs Ready Ready Preview Aug 29, 2026 1:14am

Request Review

@darkgnotic
darkgnotic enabled auto-merge August 29, 2026 01:14
@darkgnotic
darkgnotic added this pull request to the merge queue Aug 29, 2026
Merged via the queue into main with commit 9eb703f Aug 29, 2026
30 checks passed
@darkgnotic
darkgnotic deleted the darkgnotic/fix-comment-on-publication-misclassification branch August 29, 2026 01:30
darkgnotic added a commit that referenced this pull request Aug 29, 2026
…UBLICATION (#6440)

Fix the classification of schema change type when processing a `COMMENT
ON PUBLICATION` tag. Examining the previous schema change is incorrect
as in Supabase the `ALTER PUBLICATION` will be missing. So always
consider a schema change detected by `COMMENT ON PUBLICATION` as
"unknown" which will correctly initiate a backfill for new
tables/columns.

Context:
* https://rocicorp.slack.com/archives/C0B3A4P3W56/p1787956562056209
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.

1 participant