Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
fbfa7d5
Re-enable ClickHouse in CLI tests
devin-ai-integration[bot] Jan 27, 2026
dee008c
Fix prettier formatting for warehouse-type array
devin-ai-integration[bot] Jan 27, 2026
d0558f7
Merge branch 'master' into ele-5219-re-enable-clickhouse-in-cli-tests
haritamar Jan 27, 2026
35c94c1
Add clickhouse to Seed e2e dbt project step
devin-ai-integration[bot] Jan 27, 2026
bbeeecc
Merge branch 'ele-5219-re-enable-clickhouse-in-cli-tests' of https://…
devin-ai-integration[bot] Jan 27, 2026
09ed6df
Merge branch 'master' into ele-5219-re-enable-clickhouse-in-cli-tests
haritamar Jan 27, 2026
1fc1952
Fix get_elementary_database_and_schema to use actual elementary schema
devin-ai-integration[bot] Jan 27, 2026
c92b6a9
Revert "Fix get_elementary_database_and_schema to use actual elementa…
devin-ai-integration[bot] Jan 27, 2026
a3bef9c
Fix ClickHouse column alias handling in populate_test_alerts macro
devin-ai-integration[bot] Jan 27, 2026
5a7efb8
Use .get() for all field accesses in populate_test_alerts to handle C…
devin-ai-integration[bot] Jan 27, 2026
327b0bf
Merge branch 'master' into ele-5219-re-enable-clickhouse-in-cli-tests
haritamar Jan 27, 2026
8e7e947
Revert "Use .get() for all field accesses in populate_test_alerts to …
devin-ai-integration[bot] Jan 27, 2026
bb0d159
Revert "Fix ClickHouse column alias handling in populate_test_alerts …
devin-ai-integration[bot] Jan 27, 2026
d279681
Merge branch 'ele-5219-re-enable-clickhouse-in-cli-tests' of https://…
devin-ai-integration[bot] Jan 27, 2026
1ce294e
Add debug logging to identify Undefined field in populate_test_alerts
devin-ai-integration[bot] Jan 27, 2026
a34bd77
Update debug logging to use 'is defined' checks for accurate Undefine…
devin-ai-integration[bot] Jan 27, 2026
f30af4f
Add more detailed debug logging to find nested Undefined values
devin-ai-integration[bot] Jan 27, 2026
c6103be
Fix ClickHouse column alias handling with .get() fallbacks
devin-ai-integration[bot] Jan 27, 2026
021fd2a
Fix ClickHouse column name ambiguity with explicit SQL aliases
devin-ai-integration[bot] Jan 27, 2026
ef0a724
Remove DBT_EDR_DEBUG env var (debugging complete)
devin-ai-integration[bot] Jan 27, 2026
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 9 additions & 1 deletion .github/workflows/test-all-warehouses.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,15 @@ jobs:
matrix:
dbt-version: ${{ inputs.dbt-version && fromJSON(format('["{0}"]', inputs.dbt-version)) || fromJSON('[null]') }}
warehouse-type:
[postgres, snowflake, bigquery, redshift, databricks_catalog, athena]
[
postgres,
snowflake,
bigquery,
redshift,
databricks_catalog,
athena,
clickhouse,
]
uses: ./.github/workflows/test-warehouse.yml
with:
warehouse-type: ${{ matrix.warehouse-type }}
Expand Down
11 changes: 6 additions & 5 deletions .github/workflows/test-warehouse.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ on:
- databricks_catalog
- spark
- athena
- clickhouse
elementary-ref:
type: string
required: false
Expand Down Expand Up @@ -103,10 +104,10 @@ jobs:
working-directory: ${{ env.E2E_DBT_PROJECT_DIR }}
run: docker compose up -d postgres

# - name: Start Clickhouse
# if: inputs.warehouse-type == 'clickhouse'
# working-directory: ${{ env.E2E_DBT_PROJECT_DIR }}
# run: docker compose up -d clickhouse
- name: Start Clickhouse
if: inputs.warehouse-type == 'clickhouse'
working-directory: ${{ env.E2E_DBT_PROJECT_DIR }}
run: docker compose up -d clickhouse

- name: Setup Python
uses: actions/setup-python@v5
Expand Down Expand Up @@ -161,7 +162,7 @@ jobs:
- name: Seed e2e dbt project
working-directory: ${{ env.E2E_DBT_PROJECT_DIR }}
if: inputs.warehouse-type == 'postgres' || inputs.generate-data
if: inputs.warehouse-type == 'postgres' || inputs.warehouse-type == 'clickhouse' || inputs.generate-data
run: |
python generate_data.py
dbt seed -f --target "${{ inputs.warehouse-type }}"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
{% set test_alerts = [] %}
{% set raw_test_alerts_agate = run_query(elementary_cli.populate_test_alerts_query(days_back)) %}
{% set raw_test_alerts = elementary.agate_to_dicts(raw_test_alerts_agate) %}

{% for raw_test_alert in raw_test_alerts %}
{% set test_type = raw_test_alert.alert_type %}
{% set status = raw_test_alert.status | lower %}
Expand Down Expand Up @@ -138,25 +139,26 @@
failed_tests.test_execution_id,
failed_tests.test_unique_id,
failed_tests.model_unique_id,
failed_tests.database_name,
{# Explicit aliases for columns that exist in both failed_tests and tests tables to avoid ClickHouse column name ambiguity #}
failed_tests.database_name as database_name,
failed_tests.detected_at,
{{ elementary.edr_current_timestamp() }} as created_at,
failed_tests.schema_name,
failed_tests.schema_name as schema_name,
failed_tests.table_name,
failed_tests.column_name,
failed_tests.test_type as alert_type,
failed_tests.sub_type,
failed_tests.test_results_description as alert_description,
failed_tests.owners,
failed_tests.tags,
failed_tests.tags as tags,
failed_tests.test_results_query as alert_results_query,
failed_tests.other,
failed_tests.test_name,
failed_tests.test_short_name,
failed_tests.test_params,
failed_tests.severity,
failed_tests.status,
failed_tests.result_rows,
failed_tests.test_params as test_params,
failed_tests.severity as severity,
failed_tests.status as status,
failed_tests.result_rows as result_rows,
tests.meta as test_meta,
tests.description as test_description,
artifacts_meta.meta as model_meta,
Expand Down
Loading