You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "zhangxinyao88 (via GitHub)" <gi...@apache.org> on 2026/08/22 18:27:46 UTC
[PR] Add tests for DataFusion base classes [airflow]
zhangxinyao88 opened a new pull request, #71983:
URL: https://github.com/apache/airflow/pull/71983
Add focused tests for the Common SQL DataFusion base abstractions:
- cover bucket extraction from valid and invalid object-storage paths
- verify format handlers retain their data-source configuration
- verify the DataFusion exception types share the expected Airflow exception hierarchy
Remove the corresponding entries from `OVERLOOKED_TESTS` now that both test modules exist.
related: #35442
Testing:
- `breeze run pytest providers/common/sql/tests/unit/common/sql/datafusion/test_base.py providers/common/sql/tests/unit/common/sql/datafusion/test_exceptions.py -xvs`
- `breeze run pytest airflow-core/tests/unit/always/test_project_structure.py::TestProjectStructure::test_providers_modules_should_have_tests -xvs`
- Airflow pre-commit and commit-message hooks
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes (Codex)
Generated-by: Codex following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
---
Drafted-by: Codex GPT-5; reviewed by @zhangxinyao88 before posting
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add tests for DataFusion base classes [airflow]
Posted by "uros-b (via GitHub)" <gi...@apache.org>.
uros-b commented on PR #71983:
URL: https://github.com/apache/airflow/pull/71983#issuecomment-5392565553
Thank you @zhangxinyao88!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add tests for DataFusion base classes [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #71983:
URL: https://github.com/apache/airflow/pull/71983
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add tests for DataFusion base classes [airflow]
Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on PR #71983:
URL: https://github.com/apache/airflow/pull/71983#issuecomment-5602314776
Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org