Skip to content

refactor: migrate to smaller example for Snowflake#767

Merged
farazdagi merged 1 commit into
mainfrom
ref/snowflake-example
May 22, 2026
Merged

refactor: migrate to smaller example for Snowflake#767
farazdagi merged 1 commit into
mainfrom
ref/snowflake-example

Conversation

@farazdagi
Copy link
Copy Markdown
Contributor

Description

  • As agreed, replacing the huge Snowflake TUI app with a single file example, matching the other destinations
  • Add cargo x seed: provisions a Postgres database with test tables and sample data (so that it is trivial to actually see examples in action)
  • Add cargo x example <name>: runs any destination example, auto-injecting credentials from TESTS_* env vars
  • Expand .env.example with setup instructions for all destinations (BigQuery, ClickHouse, Snowflake)

Usage

# One-time: seed a test database
cargo x seed

# Run any example 
source .env
cargo x example snowflake
cargo x example bigquery
cargo x example clickhouse

# Override defaults
cargo x example snowflake -- --db-name mydb --max-batch-fill-duration-ms 1000

# Re-seed with --force
cargo x seed --force --rows 5000

What was removed

The Snowflake TUI app (src/bin/snowflake/{main,tui,state,commands,logging}.rs) and snowflake_loadgen.rs have been extracted to a standalone repo. The simplified example follows the same pattern as all other destinations.

@farazdagi farazdagi self-assigned this May 22, 2026
@farazdagi farazdagi requested a review from a team as a code owner May 22, 2026 13:29
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 72.885% (-0.3%) from 73.187% — ref/snowflake-example into main

@farazdagi farazdagi merged commit 169ff7f into main May 22, 2026
18 checks passed
@farazdagi farazdagi deleted the ref/snowflake-example branch May 22, 2026 13:54
ttatsato pushed a commit to ttatsato/supabase-etl that referenced this pull request May 25, 2026
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.

3 participants