Skip to content

load, crossmatch, statistics: the child tables are logged - #87

Merged
rusholme merged 1 commit into
mainfrom
logged-tables
Oct 15, 2026
Merged

rusholme merged 1 commit into
mainfrom
logged-tables

Conversation

@rusholme

Copy link
Copy Markdown
Collaborator

rapid migration 20261013-01 (rebuild a88b6152, Ben's ruling of 2026-10-13) makes every table LOGGED and removes SET UNLOGGED from create_sources_child_table, create_field_object_tables and create_astroobjectsmeta_child_table. Three pages still described those tables as unlogged.

  • system/load.md: the sources child table is logged, unlike dev's. One sentence gives the reason: after rapid-db stopped uncleanly on 2026-10-02, crash recovery emptied the unlogged tables while their result sets still read complete.
  • system/crossmatch.md: astroobjects_<field> and merges_<field> are logged, unlike dev's.
  • system/statistics.md: astroobjectsmeta_<field> is logged, unlike dev's.

Local checks: check-public-safety.sh OK; sphinx-build -W html passes; acceptance.sh passes. Linkcheck was not run; the change adds no links.

Not in this PR: system/decisions.md:112 still says "rapid_rebuild is disposable", for a database dropped on 2026-10-13. That entry waits on the catalogue-of-record ruling.

🤖 Generated with Claude Code

rapid migration 20261013-01 (Ben, 2026-10-13) sets every table LOGGED and
drops SET UNLOGGED from the three child-table functions, after crash
recovery on 2026-10-02 emptied the unlogged source and object tables
under result sets that still read complete. The three pages described
the tables as unlogged; they now say logged, where dev's are unlogged,
and load gives the reason once.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@rusholme
rusholme merged commit 5b8e109 into main Oct 15, 2026
2 checks passed
@rusholme
rusholme deleted the logged-tables branch October 15, 2026 05:01
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