Skip to content

Conversation

@sc250072
Copy link
Contributor

This PR introduces a new feature in the Airflow Teradata provider to support execution of Teradata DDL scripts as part of Airflow DAGs. The new operator enhances integration with Teradata by enabling flexible orchestration of DDL workloads both locally and remotely via SSH.

Key Features:

  • Executes DDL SQL statements (CREATE, ALTER, DROP, etc.)
  • Works with single statements or batches of multiple DDL operations
  • Integrates with Airflow's connection management for secure database access
  • Provides comprehensive logging of execution results
  • Supports both local and remote execution via SSH

🛠️ Additional Enhancements:
Includes utility functions for:

File validation and encoding checks


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

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

Generally fine, but you shoul rewrite the AirflowException behaviour following our latest discussions - https://lists.apache.org/thread/5rv4tz0oc27bgr4khx0on0jz8fpxvh55

@sc250072
Copy link
Contributor Author

Generally fine, but you shoul rewrite the AirflowException behaviour following our latest discussions - https://lists.apache.org/thread/5rv4tz0oc27bgr4khx0on0jz8fpxvh55

We rewritten the airflow exception with specific exception in this PR code. Could you please review and let us know if any further updates are needed?.

@sc250072
Copy link
Contributor Author

Generally fine, but you shoul rewrite the AirflowException behaviour following our latest discussions - https://lists.apache.org/thread/5rv4tz0oc27bgr4khx0on0jz8fpxvh55

We rewritten the airflow exception with specific exception in this PR code. Could you please review and let us know if any further updates are needed?.

@potiuk we have rewritten the airflow exception with specific exception. Could you please review

@potiuk potiuk merged commit e4f07e3 into apache:main Nov 24, 2025
83 checks passed
Copilot AI pushed a commit to jason810496/airflow that referenced this pull request Dec 5, 2025
…on Teradata databases using TTU tbuild utility (apache#56675)

* override sqlalchemy_url and get_uri for SQLAlchemy

* IDE-24617 TPT DDL Operator implementation

* refactor: Replace AirflowException with built-in exceptions

* Fix provider.yaml

* Add match parameter to pytest.raises for PT011 compliance

* Replace preexec_fn with start_new_session for thread-safe subprocess execution

---------

Co-authored-by: Satya Gopu <satyanarayanareddy.gopu@teradata.com>
Co-authored-by: Satya Gopu <151805211+snreddygopu@users.noreply.github.com>
itayweb pushed a commit to itayweb/airflow that referenced this pull request Dec 6, 2025
…on Teradata databases using TTU tbuild utility (apache#56675)

* override sqlalchemy_url and get_uri for SQLAlchemy

* IDE-24617 TPT DDL Operator implementation

* refactor: Replace AirflowException with built-in exceptions

* Fix provider.yaml

* Add match parameter to pytest.raises for PT011 compliance

* Replace preexec_fn with start_new_session for thread-safe subprocess execution

---------

Co-authored-by: Satya Gopu <satyanarayanareddy.gopu@teradata.com>
Co-authored-by: Satya Gopu <151805211+snreddygopu@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants