-
Notifications
You must be signed in to change notification settings - Fork 214
feat: add DuckDB, Trino, Dremio & Spark support to CI and CLI #2135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
98 commits
Select commit
Hold shift + click to select a range
388c791
feat: add DuckDB, Trino, Dremio & Spark support to CI and CLI (part 1)
devin-ai-integration[bot] 63a6bc1
feat: add Docker startup steps for Trino, Dremio, Spark in CI workflow
devin-ai-integration[bot] 04b80ad
feat: add DuckDB, Trino, Dremio, Spark profile targets
devin-ai-integration[bot] 16665b8
feat: add Trino Iceberg catalog config for CI testing
devin-ai-integration[bot] 6e520fb
feat: add Spark Hive metastore config for CI testing
devin-ai-integration[bot] 9351fbc
feat: add Dremio setup script for CI testing
devin-ai-integration[bot] 0836bbf
feat: add Trino, Dremio, Spark Docker services to docker-compose.yml
devin-ai-integration[bot] 40b2c74
fix: address DuckDB, Spark, and Dremio CI test failures
devin-ai-integration[bot] bac36f9
fix: use Nessie catalog source for Dremio instead of plain S3
devin-ai-integration[bot] 3682f1c
feat: add seed caching for Docker-based adapters in CI
devin-ai-integration[bot] 7969949
fix: move seed cache restore before Docker service startup
devin-ai-integration[bot] 0d00cfa
fix: add docker-compose.yml to seed cache key and fail-fast readiness…
devin-ai-integration[bot] 7213534
fix: convert ClickHouse bind mount to named Docker volume for seed ca…
devin-ai-integration[bot] 25b81ff
ci: temporarily use dbt-data-reliability fix branch for Trino/Spark s…
devin-ai-integration[bot] c40da20
fix: stop Docker containers before archiving seed cache volumes
devin-ai-integration[bot] 03c8c60
fix: add readiness wait after restarting Docker containers for seed c…
devin-ai-integration[bot] 2b1c7c5
fix: use Trino starting:false check for proper readiness detection
devin-ai-integration[bot] 6459984
fix: add Hive Metastore readiness check after container restart for T…
devin-ai-integration[bot] af20d30
fix: Dremio CI - batched seed materialization, single-threaded seedin…
devin-ai-integration[bot] dfa315e
fix: Dremio CI - single-threaded dbt run/test, fix cross-schema seed …
devin-ai-integration[bot] 74368de
fix: revert reserved word quoting, use Dremio-specific expected failu…
devin-ai-integration[bot] 7586f6e
fix: rename reserved word columns (min/max/sum/one) to avoid Dremio S…
devin-ai-integration[bot] b6f51e3
fix: always run seed step for all adapters (cloud adapters need fresh…
devin-ai-integration[bot] 0165356
fix: Dremio generate_schema_name - use default_schema instead of root…
devin-ai-integration[bot] 71fbaf7
fix: Dremio - put seeds in default schema to avoid cross-schema refer…
devin-ai-integration[bot] a1c8be2
feat: external seed loading for Dremio and Spark via MinIO/CSV instea…
devin-ai-integration[bot] 2db8cc9
fix: format load_seeds_external.py with black, remove unused imports
devin-ai-integration[bot] 99945c8
fix: use --entrypoint /bin/sh for minio/mc docker container to enable…
devin-ai-integration[bot] dfac673
refactor: extract external seeders into classes with click CLI
devin-ai-integration[bot] 13936c1
fix: black/isort formatting in dremio.py
devin-ai-integration[bot] 6b835c9
fix: read Dremio credentials from dremio-setup.sh for external seeder
devin-ai-integration[bot] 094841b
fix: regex to handle escaped quotes in dremio-setup.sh for credential…
devin-ai-integration[bot] fd4738b
fix: use COPY INTO for Dremio seeds, skip Spark seed caching
devin-ai-integration[bot] eaca0ab
fix: add file_format delta for Spark models in e2e dbt_project.yml
devin-ai-integration[bot] 039281c
fix: Dremio S3 source - use compatibilityMode, rootPath=/, v3 Catalog…
devin-ai-integration[bot] 6b77609
fix: Dremio root_path double-nesting + Spark CLI file_format delta
devin-ai-integration[bot] 11379cf
fix: Dremio Space architecture - views in Space, seeds in Nessie data…
devin-ai-integration[bot] a5449ad
style: apply black formatting to dremio.py
devin-ai-integration[bot] 703d183
fix: restore dremio.py credential extraction from dremio-setup.sh
devin-ai-integration[bot] b7e3ee0
fix: use enterprise_catalog_namespace for Dremio to avoid Nessie vers…
devin-ai-integration[bot] b8790d6
fix: restore dremio__generate_schema_name delegation for correct Ness…
devin-ai-integration[bot] 692b494
fix: flatten Dremio seed schema to single-level Nessie namespace
devin-ai-integration[bot] 6cbc6a8
fix: avoid typos pre-commit false positive on SOURCE plural
devin-ai-integration[bot] 2bead60
fix: create Nessie namespace via REST API + refresh source metadata
devin-ai-integration[bot] 9b6bb1c
style: apply black formatting to Nessie namespace methods
devin-ai-integration[bot] a325dcc
fix: improve Nessie namespace creation + force Dremio catalog discovery
devin-ai-integration[bot] 3a0f781
fix: force NessieSource metadata re-scan via Catalog API policy update
devin-ai-integration[bot] 9752026
fix: use USE BRANCH main for Dremio Nessie version context resolution
devin-ai-integration[bot] 25576cb
fix: put Dremio seeds in same Nessie namespace as models to fix VDS v…
devin-ai-integration[bot] 9f5623e
fix: use CREATE FOLDER + ALTER SOURCE REFRESH for Dremio metadata vis…
devin-ai-integration[bot] 149f7b8
fix: skip Docker restart for Dremio to preserve Nessie metadata cache
devin-ai-integration[bot] 0415db5
fix: resolve Dremio edr monitor duplicate keys + exclude ephemeral mo…
devin-ai-integration[bot] 196be41
fix: add continue-on-error for Dremio edr steps (dbt-core 1.11 compat)
devin-ai-integration[bot] 4fba3c4
fix: revert temporary dbt-data-reliability branch pin (PR #948 merged)
devin-ai-integration[bot] dcfa5e6
refactor: address PR review - ref syntax, healthchecks, external scripts
devin-ai-integration[bot] 36fb6ff
refactor: parameterize Docker credentials via environment variables
devin-ai-integration[bot] 34f9160
style: fix prettier formatting for docker-compose.yml healthchecks
devin-ai-integration[bot] f489915
fix: increase Docker healthcheck timeouts for CI and fix Spark volume…
devin-ai-integration[bot] 849c017
fix: increase dremio-minio healthcheck retries to 60 with start_perio…
devin-ai-integration[bot] e7de065
fix: use bash TCP check for healthchecks (curl/nc missing in MinIO 20…
devin-ai-integration[bot] 04aa215
fix: align dremio-setup.sh default password with docker-compose (drem…
devin-ai-integration[bot] 0fad79f
fix: resolve dremio.py credential extraction from shell variable defa…
devin-ai-integration[bot] 6c0b313
fix: increase hive-metastore healthcheck retries to 60 with 60s start…
devin-ai-integration[bot] a8214bd
fix: wait for dremio-setup to complete before proceeding (use --exit-…
devin-ai-integration[bot] 4518e15
fix: add nessie dependency to dremio-setup so NessieSource creation s…
devin-ai-integration[bot] a790a88
fix: use ghcr.io registry for nessie image (no longer on Docker Hub)
devin-ai-integration[bot] d06c1e3
fix: add continue-on-error for Dremio edr steps (dbt-core 1.11 ref() …
devin-ai-integration[bot] e450aa3
Merge remote-tracking branch 'origin/master' into devin/ELE-5266-1772…
devin-ai-integration[bot] 7328c48
fix: remove continue-on-error for Dremio edr steps (ref() override no…
devin-ai-integration[bot] 1182be3
fix: use dot-separated Nessie namespace for Dremio elementary profile
devin-ai-integration[bot] 454d7be
fix: rename 'snapshots' CTE to avoid Dremio reserved keyword conflict
devin-ai-integration[bot] 88b87c8
fix: quote 'filter' column to avoid Dremio reserved keyword conflict
devin-ai-integration[bot] 820fd32
fix: make 'filter' column quoting Dremio-specific to avoid Snowflake …
devin-ai-integration[bot] 0a934d3
fix: override dbt-dremio dateadd to handle integer interval parameter
devin-ai-integration[bot] 897bf50
fix: remove 'select' prefix from dateadd override to avoid $SCALAR_QU…
devin-ai-integration[bot] abd09de
fix: strip Z timezone suffix from Dremio timestamps to avoid GandivaE…
devin-ai-integration[bot] 7351b30
fix: use double quotes in dbt_project.yml for prettier compatibility
devin-ai-integration[bot] db901e5
fix: also replace T separator with space in Dremio timestamp cast
devin-ai-integration[bot] f0b3b09
fix: use targeted regex for T separator to avoid replacing T in non-t…
devin-ai-integration[bot] b4f7a3e
fix: quote 'filter' reserved keyword in get_source_freshness_results …
devin-ai-integration[bot] 0e132ee
fix: quote Dremio reserved keywords row_number and count in SQL aliases
devin-ai-integration[bot] 3158fe6
refactor: use elementary.escape_reserved_keywords() for Dremio reserv…
devin-ai-integration[bot] 8a9596d
chore: revert temporary dbt-data-reliability branch pin (PR #955 merged)
devin-ai-integration[bot] 5e97cec
fix: resolve 'Column unique_id is ambiguous' error in Dremio joins
devin-ai-integration[bot] 00afc47
fix: qualify invocation_id column reference to resolve ambiguity in O…
devin-ai-integration[bot] d48a335
Merge branch 'master' into devin/ELE-5266-1772368864-code-ci
haritamar efaab28
fix: address CodeRabbit review comments
devin-ai-integration[bot] 19652db
Merge branch 'devin/ELE-5266-1772368864-code-ci' of https://git-manag…
devin-ai-integration[bot] a63ce6b
style: fix black formatting in dremio.py and spark.py
devin-ai-integration[bot] 2263199
fix: address CodeRabbit bugs - 409 fallback and stale empty tables
devin-ai-integration[bot] becc9a1
fix: address remaining CodeRabbit CI comments
devin-ai-integration[bot] a5dcef9
fix: clarify Spark seeder pyhive dependency
devin-ai-integration[bot] a50645c
fix: address CodeRabbit review round 3 - cleanup and hardening
devin-ai-integration[bot] 05c57a3
fix: correct isort import order in dremio.py
devin-ai-integration[bot] 114dd3f
fix: restore continue-on-error on dbt test step (many e2e tests are d…
devin-ai-integration[bot] 856ddfa
Merge remote-tracking branch 'origin/master' into devin/ELE-5266-1772…
devin-ai-integration[bot] 71d11e7
fix: remove Dremio dateadd and cast_column overrides now handled by d…
devin-ai-integration[bot] 3ecb6f4
fix: remove dremio_target_database override now handled by dbt-data-r…
devin-ai-integration[bot] File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
30 changes: 15 additions & 15 deletions
30
elementary/monitor/dbt_project/macros/get_models_latest_invocations_data.sql
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.