baolsen commented on a change in pull request #7541: [AIRFLOW-6822] AWS hooks
should cache boto3 client
URL: https://github.com/apache/airflow/pull/7541#discussion_r385542156
##########
File path: airflow/providers/amazon/aws/hooks/base_aws.py
##########
@@ -47,11 +48,29 @@ class AwsBaseHook(BaseHook):
:param verify: Whether or not to verify SSL certificates.
https://boto3.amazonaws.com/v1/documentation/api/latest/reference/core/session.html
:type verify: str or bool
+ :param str region_name: AWS Region name to use. If this is None then the
default boto3
+ behaviour is used.
+ :param str client_type: boto3 client_type used when creating
boto3.client(). For
+ example, 's3', 'emr', etc. Provided by specific hooks for these
clients which
+ subclass AwsBaseHook.
+ :param str resource_type: boto3 resource_type used when creating
boto3.resource(). For
+ example, 's3'. Provided by specific hooks for these resources which
+ subclass AwsBaseHook.
Review comment:
Resolved, thanks.
I preferred the conciseness of `:param <type> <var>: xyz`
But have changed back to `:param` + `:type`
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
