sjyangkevin commented on issue #58381:
URL: https://github.com/apache/airflow/issues/58381#issuecomment-3736631200

   >I also noticed ti_update_state is updating the task state, wonder if it can 
be the place I start to look into.
   >Yes
   
   Thanks for the feedback. I will try out adding the logging into 
`ti_update_state`. Meanwhile, I am also looking into the 2.11.0 code and trying 
to better understand how those states are actually logged, I feel that would 
also help me to find out the proper place to implement the fix.
   
   > `action_logging` "might" not be the thing we want to reuse. This pattern 
is not used in execution_api (only in core_api for the time being)
   
   Yeah, agree on this. From my understanding, it looks more like to log user 
actions instead of system-generated events (e.g., task lifecycle state 
transition).


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