You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "jscheffl (via GitHub)" <gi...@apache.org> on 2024/06/08 20:33:17 UTC
[PR] Move Post Execution Log Grouping behind Exception Print [airflow]
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:
...and you needed to un-fold to see it:
After - exception is moved in front of log group:
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Move Post Execution Log Grouping behind Exception Print [airflow]
Posted by "eladkal (via GitHub)" <gi...@apache.org>.
eladkal merged PR #40146:
URL: https://github.com/apache/airflow/pull/40146
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org