Results 6 comments of lyuyunbo

Hi @potiuk, I've added a PR to address your [previous discussion](https://github.com/apache/airflow/discussions/29302) with [MattiaGallegati](https://github.com/MattiaGallegati). Would like to have your review on this. Thanks.

> I find it as a useful optional feature, but it absolutely cannot be enabled by default. It should be quite a bit more generic also in order to be...

> Can you add the test cases for this change? Change has been made in the new PR. Further review will be very welcomed and appreciated. @dirrao

> conflicts to resolve when rebasing. Resolved.

> Still try/finally not solved Can you be more specific please @potiuk? Because we indeed added `_restore_log_formatter` into the except clause, which will revert the changes if formatters are not...

> ``` > change() > try: > do stuff > finally: > restore() > ``` Ha! I misunderstood which try-catch block you were mentioning before. Updated, this is indeed a...