Taragolis commented on PR #27947: URL: https://github.com/apache/airflow/pull/27947#issuecomment-1404455178
Thin `boto3` client wrapper it is a Hook based on AwsBaseHook without extending positional and key-word arguments created. IMHO and personal thought: "The last think that we want it is add additional arguments in Hook and make them Thick Wrapper". There is couple of hooks which accept additional arguments and it it turned into the hell to support, test and use them. You could check [Redshift.Client](https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/redshift.html#Redshift.Client) and found that not every method required to ClusterIdentifier as well as other fields you described. -- 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]
