Skip to content

Feat/dbt starburst#20

Open
pmbrull wants to merge 20 commits into
mainfrom
feat/dbt-starburst
Open

Feat/dbt starburst#20
pmbrull wants to merge 20 commits into
mainfrom
feat/dbt-starburst

Conversation

@pmbrull
Copy link
Copy Markdown
Member

@pmbrull pmbrull commented May 10, 2026

Summary

Changes

SDKs Affected

  • Python SDK
  • TypeScript SDK
  • Java SDK
  • Rust CLI
  • n8n Node
  • Documentation
  • CI/Build

Checklist

Required

  • Code follows the project's style guidelines (make lint passes)
  • All existing tests pass (make test-all passes)
  • New code has appropriate test coverage
  • Changes are consistent across all affected SDKs

If Adding New Functionality

  • Added to all relevant SDKs (not just one)
  • Added unit tests for new code
  • Updated type definitions where applicable

If Fixing a Bug

  • Added test that reproduces the bug
  • Test now passes with the fix

If Changing API Behavior

  • Changes are backward compatible (or breaking change is documented)
  • Updated any affected documentation

Testing

Local testing:

make lint      # ✅ Pass
make test-all  # ✅ Pass

Integration testing: (if applicable)

  • Tested against real Metadata instance
  • Streaming works correctly
  • Error handling works as expected

Additional Notes

pmbrull and others added 20 commits May 9, 2026 13:10
14 CSVs exported from the running PG demo container via the
export-raw-to-seeds.sh script. Total ~320KB. Used by `dbt seed
--target starburst` to populate the Iceberg raw_* schemas.
…ility

- hours_between macro replaces extract(epoch from interval) in tickets
- day_of_week_iso macro replaces extract(dow ...) in fct_orders (now ISO 1-7)
- Add install-dbt-starburst Make target

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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