Skip to content

Conversation

@mrhallak
Copy link

@mrhallak mrhallak commented Jan 15, 2026

Summary

  • Rename databricks_dlt task type to declarative_pipeline to match generated task YAML files from the DLT task generator plugin in dataeng-airflow-dags

Files Renamed

Old Name New Name
databricks_dlt_task.py declarative_pipeline_task.py
databricks_dlt_creator.py declarative_pipeline_creator.py
test_databricks_dlt_task.py test_declarative_pipeline_task.py
test_databricks_dlt_creator.py test_declarative_pipeline_creator.py
databricks_dlt_task.yaml declarative_pipeline_task.yaml

Changes

  • Class names: DatabricksDLTTaskDeclarativePipelineTask
  • Class names: DatabricksDLTCreatorDeclarativePipelineCreator
  • ref_name: "databricks_dlt""declarative_pipeline"
  • Updated imports in factory files

Related

  • dataeng-airflow-dags PR #10644 (DLT plugin architecture refactor)

Test plan

  • All 84 tests pass (4 graphviz-related failures are pre-existing)
  • Task type properly registered via TaskFactory
  • Operator creator properly registered via OperatorFactory

Rename task type and operator creator to match the generated task YAML
files from the DLT task generator plugin.

Files renamed:
- databricks_dlt_task.py -> declarative_pipeline_task.py
- databricks_dlt_creator.py -> declarative_pipeline_creator.py
- test_databricks_dlt_task.py -> test_declarative_pipeline_task.py
- test_databricks_dlt_creator.py -> test_declarative_pipeline_creator.py
- databricks_dlt_task.yaml -> declarative_pipeline_task.yaml

Changes:
- Class names: DatabricksDLTTask -> DeclarativePipelineTask
- Class names: DatabricksDLTCreator -> DeclarativePipelineCreator
- ref_name: "databricks_dlt" -> "declarative_pipeline"
- Updated imports in factory files
@mrhallak mrhallak marked this pull request as ready for review January 15, 2026 15:44
@mrhallak mrhallak requested a review from a team as a code owner January 15, 2026 15:44
@mrhallak mrhallak merged commit 0e92e9c into master Jan 15, 2026
2 checks passed
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