Skip to content

refactor: check DocDB v1 names against v2 - #65

Merged
seanmcculloch merged 2 commits into
mainfrom
record-consistency-v1-coverage
Oct 13, 2026
Merged

seanmcculloch merged 2 commits into
mainfrom
record-consistency-v1-coverage

Conversation

@seanmcculloch

@seanmcculloch seanmcculloch commented Oct 1, 2026 •

Copy link
Copy Markdown
Collaborator

What changed

Add a docdb_v1 source and a check that fails v1 records with no exact name match in v2. The source sweeps v1 by _id and refetches only records whose last_modified changed since the previous snapshot.

Review notes

Matched in time, the failures overlap almost entirely with metadata_upgrade's not-upgraded rows. Keep it, re-source it from that table, or drop it?

@seanmcculloch
seanmcculloch force-pushed the record-consistency-v1-coverage branch 2 times, most recently from 2a99d7f to 443dd7a Compare October 1, 2026 17:31
@seanmcculloch
seanmcculloch added this pull request to stack #67 October 1, 2026 17:31
@seanmcculloch
seanmcculloch force-pushed the record-consistency-v1-coverage branch 2 times, most recently from 9e641f5 to 5f431c6 Compare October 1, 2026 18:23
@seanmcculloch
seanmcculloch marked this pull request as ready for review October 1, 2026 18:53
@seanmcculloch
seanmcculloch requested a review from dbirman October 1, 2026 18:54
@seanmcculloch seanmcculloch changed the title feat: check DocDB v1 names against v2 refactor: check DocDB v1 names against v2 Oct 1, 2026
Base automatically changed from record-consistency-checks to main October 7, 2026 05:26
@dbirman
dbirman force-pushed the record-consistency-v1-coverage branch 2 times, most recently from 68396ca to 8cebd6f Compare October 7, 2026 05:28

@dbirman dbirman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good

@seanmcculloch
seanmcculloch force-pushed the record-consistency-v1-coverage branch from 8cebd6f to 1fd319e Compare October 13, 2026 22:39
@seanmcculloch
seanmcculloch merged commit 9e9f1e6 into main Oct 13, 2026
3 checks passed
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