dstandish opened a new pull request, #37857:
URL: https://github.com/apache/airflow/pull/37857

   …skInstance
   
   Airflow TIs are assumed to log to the "airflow.task" logger.  TaskInstance 
was configuring this in the "reconstructor", a sqlalchemy thing that is 
processed on loading from DB.  But there's no equivalent for Pydantic models, 
and when running a task with TaskInstancePydantic, the wrong logger was used 
(fully qualified class name).  When looking for a solution, I saw that 
LoggingMixin will check `_logger_name` and if set will use it.  This works for 
the pydantic model and seems better for TI anyway so I updated both.
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to