turbaszek commented on a change in pull request #9631:
URL: https://github.com/apache/airflow/pull/9631#discussion_r453187575
##########
File path: airflow/models/taskinstance.py
##########
@@ -1132,7 +1133,8 @@ def signal_handler(signum, frame): # pylint:
disable=unused-argument
self._run_execute_callback(context, task)
# Execute the task
- result = self._execute_task(context, task_copy)
+ with set_current_context(context):
Review comment:
Hm, I think the `get_current_context` is only for "functional dags"
(using `@task` to decorate functions) and there's no support for pre a post
executes. So in my opinion only `execute` should be wrapped in this context.
@casassg
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
