Skip to content

Enhanced the BTEQ operator to ensure platform independence. #52252

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 2 commits into from
Jun 26, 2025

Conversation

sc250072
Copy link
Contributor


This PR introduces changes to the BTEQ operator to ensure it can function consistently across different platforms. The updates abstract platform-specific dependencies and streamline execution, improving portability and compatibility in various environments.

Key Changes:

  • Removed hardcoded platform-dependent logic.
  • Replaced OS-specific commands with cross-platform alternatives.
  • Added validation to detect platform and apply appropriate configurations dynamically.
  • Refactored script paths and file handling to be OS-agnostic.

Testing:

  • Verified functionality on Linux and Windows environments.
  • Existing unit/integration tests passed successfully.
  • Added/updated tests to cover platform-specific scenarios.

https://github.com/Teradata/airflow/actions/runs/15868894429 - Unit tests execution status

https://github.com/Teradata/airflow/actions/runs/15868895659 - Documentation status

https://github.com/Teradata/airflow/actions/runs/15868897290 - System Tests execution status

https://teradata.github.io/airflow/ - System Dashboard

^ 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.

sc250072 and others added 2 commits June 24, 2025 20:41
* OS platform dependent code changed to platform independent

* mac platform verified and adjusted code to work with zsh and normal shell

---------

Co-authored-by: Satish Ch <[email protected]>
@potiuk potiuk merged commit 79d5740 into apache:main Jun 26, 2025
74 checks passed
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.

2 participants