Skip to content

Comments

Update views on auto-migrate#3631

Merged
joshua-spacetime merged 5 commits intomasterfrom
shub/view-auto-migration
Nov 12, 2025
Merged

Update views on auto-migrate#3631
joshua-spacetime merged 5 commits intomasterfrom
shub/view-auto-migration

Conversation

@Shubham8287
Copy link
Contributor

@Shubham8287 Shubham8287 commented Nov 11, 2025

Description of Changes

Improves auto-migration support for views by minimizing the cases where we must disconnect clients.

Before this patch, any schema compatible view update and even no view updates at all would require us to disconnect clients, because we had to assume that the view was modified thereby rendering its currently materialized result set stale and out of date.

Patch adds a code to re-evaluate all views whose entry is in st_view_sub and make update_database to start using commit_and_broadcast_event so that now we only have to disconnect clients for incompatible view updates or dropped views.

API and ABI breaking changes

NA

Expected complexity level and risk

2

Testing

Added smoketests.

Copy link
Collaborator

@joshua-spacetime joshua-spacetime left a comment

Choose a reason for hiding this comment

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

Nice refactoring and even nicer tests!

@joshua-spacetime joshua-spacetime changed the title Views: automigration Update views on auto-migrate Nov 11, 2025
@Shubham8287 Shubham8287 force-pushed the shub/view-auto-migration branch from 51da2df to cc63327 Compare November 11, 2025 19:09
@Shubham8287 Shubham8287 added this pull request to the merge queue Nov 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 11, 2025
@joshua-spacetime joshua-spacetime added this pull request to the merge queue Nov 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 12, 2025
@joshua-spacetime joshua-spacetime added this pull request to the merge queue Nov 12, 2025
Merged via the queue into master with commit 0c6641d Nov 12, 2025
24 of 25 checks passed
@Shubham8287 Shubham8287 deleted the shub/view-auto-migration branch November 12, 2025 05:19
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