Skip to content

feat: improve Dagster checks and test fixtures - #759

Merged
kvlonge merged 4 commits into
mainfrom
fix/dagster-check-streaming
Sep 21, 2026
Merged

kvlonge merged 4 commits into
mainfrom
fix/dagster-check-streaming

Conversation

@kvlonge

@kvlonge kvlonge commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Why

Standalone SQLBuild check commands buffered Dagster check results until process completion because live checks incorrectly required an invocation-local asset materialization. SQL test fixtures also repeated typed-null projections and verbose zero-row schemas despite authoritative relation metadata.

Changes

  • stream standalone audit, check, and test results to Dagster without a same-process materialization
  • preserve materialization ordering for build-style commands and normalize Dagster metadata values
  • infer bare NULL AS column types for expected outputs and mocked relations
  • add schema-aware SELECT * FROM __empty_fixture() support
  • safely remove redundant typed-null mock columns with sqb format

Verification

  • Dagster resource tests: 60 passed
  • focused Dagster integration regression: passed
  • fixture compiler, planner, lint, integration, and DuckDB E2E tests: 102 passed
  • make check-ci
  • public repository hygiene scan

Tracking: CHI-320, CHI-321

@kvlonge kvlonge changed the title Stream standalone SQLBuild checks to Dagster fix: stream standalone checks to Dagster Sep 21, 2026
@kvlonge kvlonge changed the title fix: stream standalone checks to Dagster feat: improve Dagster checks and test fixtures Sep 21, 2026
@kvlonge
kvlonge merged commit a5923fa into main Sep 21, 2026
17 checks passed
@kvlonge
kvlonge deleted the fix/dagster-check-streaming branch September 21, 2026 04: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.

1 participant