casra-developers commented on a change in pull request #16110:
URL: https://github.com/apache/airflow/pull/16110#discussion_r640670953
##########
File path: airflow/task/task_runner/base_task_runner.py
##########
@@ -86,6 +87,10 @@ def __init__(self, local_task_job):
# pylint: disable=consider-using-with
self._error_file = NamedTemporaryFile(delete=True)
+
+ # HOTFIX: When reporting exceptions, this file was usually locked
because it was still opened by this process
+ self._error_file.close()
Review comment:
On Windows we basically just create an empty text file. Not closing it
immediately raised an exception while logging since the file was opened by
another python process.
--
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]
