-
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-5315] TaskInstance now only overwrites executor_config when explicitly told… #5926
[AIRFLOW-5315] TaskInstance now only overwrites executor_config when explicitly told… #5926
Conversation
b8dd58a
to
68ea075
Compare
68ea075
to
7bd4fce
Compare
7bd4fce
to
c5c0882
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
I won't mind if we remove Currently, it is only used in k8s executor AFAIK. Might need a note in UPDATING.md |
@kaxil where would you want to move it? |
I mean I am not able to find a use-case where we want the However, I am fine with the current change as well :) |
…explicitly told… (apache#5926) * TaskInstance now only overwrites executor_config when explicitly told to do so * flake8
… to do so
Make sure you have checked all steps below.
Jira
Description
Fixes a bug where a user is unable to update executor_configs for TaskInstances
Tests
Commits
Documentation
Code Quality
flake8