-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
[AIRFLOW-5798] Set default to None for external_task_id in ExternalTaskSensor #6431
[AIRFLOW-5798] Set default to None for external_task_id in ExternalTaskSensor #6431
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6431 +/- ##
==========================================
- Coverage 84.26% 83.39% -0.87%
==========================================
Files 627 627
Lines 36506 36549 +43
==========================================
- Hits 30760 30481 -279
- Misses 5746 6068 +322
Continue to review full report at Codecov.
|
Looking at the constructor definitions, this parameter is required. Why do you think it's optional? Should we not also change the function definitions? |
@mik-laj , this parameter is required, but if you want to wait for the entire DAG completion, not only the specific task, you should explicitly set this parameter to But should we also change the function definition is a good question, actually :-) |
That's really a (somehow) tough question. |
@vladimir-gavrilenko I created a Jira ticket because it contains changes to code. |
- fix type annotation for ExternalTaskSensor (cherry picked from commit 3fb9d49)
- fix type annotation for ExternalTaskSensor (cherry picked from commit 3fb9d49)
- fix type annotation for ExternalTaskSensor (cherry picked from commit 3fb9d49)
- fix type annotation for ExternalTaskSensor (cherry picked from commit 3fb9d49)
Make sure you have checked all steps below.
Jira
Description
Pycharm highlighted my code when I used
ExternalTaskSensor
to wait for an entire DAG completion.Two options here: use
str or None
orOptional[str]
. I can switch to the second option if necessary.Tests
No tests for docs
Commits
Documentation