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
Fix output property missing for airflow version < 2.4.0 #1385
Conversation
Codecov ReportBase: 93.87% // Head: 93.82% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1385 +/- ##
==========================================
- Coverage 93.87% 93.82% -0.05%
==========================================
Files 78 78
Lines 3673 3678 +5
Branches 355 355
==========================================
+ Hits 3448 3451 +3
- Misses 153 155 +2
Partials 72 72
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
d2b5a18
to
5dda7d5
Compare
Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
93ddb10
to
bbca90c
Compare
for more information, see https://pre-commit.ci
# Description ## What is the current behavior? <!-- Please describe the current behavior that you are modifying. --> The output was implemented in 2.4.0 according to the release notes (see [here](https://airflow.apache.org/docs/apache-airflow/stable/release_notes.html#id13)) Add output property to MappedOperator apache/airflow#25604 <!-- Issues are required for both bug fixes and features. Reference it using one of the following: closes: #ISSUE related: #ISSUE --> closes: #1359 ## What is the new behavior? <!-- Please describe the behavior or changes that are being added by this PR. --> - Catch exception for airflow version < 2.4.0 and use `XComArg(...)` instead. ## Does this introduce a breaking change? No ### Checklist - [x] Created tests which fail without the change (if possible) - [x] Extended the README / documentation, if necessary Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> (cherry picked from commit 0f5ff62)
Description
What is the current behavior?
The output was implemented in 2.4.0 according to the release notes (see here)
Add output property to MappedOperator apache/airflow#25604
closes: #1359
What is the new behavior?
XComArg(...)
instead.Does this introduce a breaking change?
No
Checklist