Skip to content

fix(config): Allow using YAML format for global config file#63

Merged
rederik76 merged 3 commits into
mainfrom
fix/yaml-config-issue
May 19, 2026
Merged

fix(config): Allow using YAML format for global config file#63
rederik76 merged 3 commits into
mainfrom
fix/yaml-config-issue

Conversation

@gopi-databricks
Copy link
Copy Markdown
Collaborator

Fixes: #50

Issue:

We hit JSONDecodeError: if we use YAML format for pipeline_configs/global.yaml
This is because dlt_pipeline_builder.py uses utility.get_json_from_file()to load the global config file which supports loading only JSON file.
Failed run:
https://e2-demo-field-eng.cloud.databricks.com/pipelines/32f8d914-b5fe-4ed8-ad58-f062897b795d/updates/5d02141c-c61e-43f3-a08d-f3fd7540cf6d?o=1444828305810485

What changes made?

A straighforward fix to use utility.load_config_file_auto will fix this problem.

Tests:

Deployed the change in internal databricks workspace and verified the run is successful
Successful run:
https://e2-demo-field-eng.cloud.databricks.com/pipelines/32f8d914-b5fe-4ed8-ad58-f062897b795d/updates/611c6014-6ce4-4c8f-a768-feeacf929ae1?o=1444828305810485

@gopi-databricks gopi-databricks requested a review from haillew May 6, 2026 07:01
@gopi-databricks gopi-databricks self-assigned this May 6, 2026
Copy link
Copy Markdown
Collaborator

@rederik76 rederik76 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@rederik76 rederik76 merged commit 2250333 into main May 19, 2026
@rederik76 rederik76 deleted the fix/yaml-config-issue branch May 19, 2026 00:00
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.

[BUG]: (YAML) legacy utility function (get_json_from_file) is being used in dlt_pipeline_builder

2 participants