dstandish commented on code in PR #43183: URL: https://github.com/apache/airflow/pull/43183#discussion_r1807102346
########## newsfragments/43183.significant.rst: ########## @@ -0,0 +1,5 @@ +Remove TaskContextLogger + +We introduced this as a way to inject messages into task logs from places +other than the task execution context. We later realized that we were better off +just using the Log table. Review Comment: this is mostly unrelated to triggerer logging. scheduler was the only place it was used. triggerer logging into webserver predates this (and was the inspiration for it) and it's unaffected by this removal. but btw this wasn't ever actually a public feature. the class itself was private (so that it could be changed easily and it was never intended to be used by users.) but there was an attr that existed on log handler that was technically public. -- 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]
