ashb commented on a change in pull request #14251:
URL: https://github.com/apache/airflow/pull/14251#discussion_r577473549
##########
File path: airflow/providers/amazon/aws/hooks/glue.py
##########
@@ -74,6 +74,7 @@ def __init__(
self.role_name = iam_role_name
self.s3_glue_logs = 'logs/glue-logs/'
kwargs['client_type'] = 'glue'
+ kwargs['region_name'] = region_name
Review comment:
There is a better fix: instead of handling `region_name` argument
ourselves, if we remove it as a named parameter we can let the parent class
constructor set `self.region_name`.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
