josh-fell commented on code in PR #24931:
URL: https://github.com/apache/airflow/pull/24931#discussion_r933609230
##########
tests/providers/google/cloud/utils/gcp_authenticator.py:
##########
@@ -133,8 +133,8 @@ def set_dictionary_in_airflow_connection(self):
extras[PROJECT_EXTRA] = self.project_extra
conn.extra = json.dumps(extras)
session.commit()
- except BaseException as ex:
- self.log.error('Airflow DB Session error: %s', str(ex))
+ except BaseException:
+ self.log.error('Airflow DB Session error.')
Review Comment:
Updating this testing util I am a little apprehensive about. I'm not sure of
the blast radius for updating. Would it be OK to not include this module in the
fixture updates?
--
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]
