Skip to content

feat: add dry-run flag to build_dataset#353

Open
Scr4tch587 wants to merge 1 commit into
kai/dry-run-storefrom
kai/dry-run-flag
Open

feat: add dry-run flag to build_dataset#353
Scr4tch587 wants to merge 1 commit into
kai/dry-run-storefrom
kai/dry-run-flag

Conversation

@Scr4tch587

Copy link
Copy Markdown
Contributor

What

Add a dry_run flag to build_dataset that selects the store and returns the produced rows.

How

  • build_dataset(dry_run=...) is the single boundary that reads the flag
  • real build → PostgresStore, returns None
  • dry run → fresh MemoryStore, returns the produced rows for the requested dataset
  • MemoryStore starts empty, so a dry run rebuilds the whole graph in isolation
  • no build lock and no cleanup — store is request-private and GC'd with the request

Why

Validate builder output end-to-end without writing to or reading from the real DB.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown

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