Skip to content

Remove stale explorer_libyear_detail refresh from matview script#3540

Merged
MoralCode merged 1 commit intoaugurlabs:mainfrom
iGufrankhan:fix/remove-stale-libyear-matview
Jan 10, 2026
Merged

Remove stale explorer_libyear_detail refresh from matview script#3540
MoralCode merged 1 commit intoaugurlabs:mainfrom
iGufrankhan:fix/remove-stale-libyear-matview

Conversation

@iGufrankhan
Copy link
Copy Markdown
Contributor

@iGufrankhan iGufrankhan commented Jan 10, 2026

Description
The scripts/control/refresh-matviews.sh script references the materialized view
augur_data.explorer_libyear_detail.

This materialized view was dropped in migration 25 and is never recreated in any subsequent migration. As a result, running the refresh script against the current schema causes it to fail when attempting to refresh a non-existent view.

This pull request removes the stale refresh statement, bringing the script back into alignment with the current database schema.

This is a minimal, one-line fix and does not change any existing functionality beyond removing the invalid reference.

This PR fixes #3538

Notes for Reviewers
Note: Should be merged after #3482 (libyear column typo fix).
No runtime testing was performed, as this change removes a stale reference identified through schema and migration inspection.

Signed commits

  • Yes, I signed my commits.

Signed-off-by: iGufrankhan <gufrankhankab123@gmail.com>
@iGufrankhan iGufrankhan requested a review from sgoggins as a code owner January 10, 2026 00:41
Copilot AI review requested due to automatic review settings January 10, 2026 00:41

This comment was marked as spam.

Copy link
Copy Markdown
Collaborator

@MoralCode MoralCode left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@shlokgilda shlokgilda left a comment

Choose a reason for hiding this comment

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

LGTM.

@MoralCode
Copy link
Copy Markdown
Collaborator

not sure why github has started being more aggressive about out of date stuff, but ill just merge this and bypass the warning

@MoralCode MoralCode merged commit 79797e6 into augurlabs:main Jan 10, 2026
15 of 16 checks passed
guptapratykshh pushed a commit to guptapratykshh/pg-augur that referenced this pull request Jan 11, 2026
…ibyear-matview

Remove stale explorer_libyear_detail refresh from matview script

Signed-off-by: guptapratykshh <pratykshgupta9999@gmail.com>
@iGufrankhan iGufrankhan deleted the fix/remove-stale-libyear-matview branch January 14, 2026 12:22
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 reference to dropped explorer_libyear_detail view in refresh-matviews.sh

4 participants