Taragolis commented on PR #34784: URL: https://github.com/apache/airflow/pull/34784#issuecomment-1750665063
>I assume followup PR would be to change AWS operators/hooks to inherit from the base classes? Yep > We need also doc update to explain how to set the botocore parameters I think it would be a good idea to add generic information here: https://airflow.apache.org/docs/apache-airflow-providers-amazon/stable/operators/index.html as soon as we change existed Operators, so them will have same sets of generic parameters: - `aws_conn_id`: Operators which intends to use AwsBaseHook already have - `region_name`: Some of operators use this parameter - `verify`: Only one use right now - `botocore_config`: None of of the operators use it right now -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org