josh-fell commented on code in PR #24931:
URL: https://github.com/apache/airflow/pull/24931#discussion_r917055812
##########
tests/utils/test_compression.py:
##########
@@ -54,8 +54,8 @@ def setUp(self):
f_bz2.writelines([header, line1, line2])
# Base Exception so it catches Keyboard Interrupt
- except BaseException as e:
- logging.error(e)
+ except BaseException:
+ logging.exception("An exception has occurred.")
Review Comment:
But perhaps going this route for core and provider modules, we get into a
situation where log handlers don't output the same (possibly useful?)
information. I'd love feedback here.
--
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]
