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

   In PR #38021 we added log grouping into log printing. We all love this 
feature but I got some negative feedback for cases where people look for a 
failed task: If you are not an Airflow expert and you look for the 
exception/eror in the logs, in many cases the exception is not printed. If the 
task execution fails in an exception the exception trace will be in the "Post 
task execution logs" group that you need to un-fold.
   
   This PR moves the "Post task execution logs" behind the exception print. 
Side effect is (due to placement) that error handling code might be called 
before (on success/skip handler will be called after/inside the group though).
   
   Before - no exception in failed task:
   
![image](https://github.com/apache/airflow/assets/95105677/038860cc-4154-41aa-8df5-6d609bf537b8)
   
   ...and you needed to un-fold to see it:
   
![image](https://github.com/apache/airflow/assets/95105677/9dbe0b69-b281-466b-b369-7da300c9ef41)
   
   After - exception is moved in front of log group:
   
![image](https://github.com/apache/airflow/assets/95105677/d0a76dd6-eadf-48ef-8181-2b9c2860685d)
   


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