ferruzzi commented on issue #24030: URL: https://github.com/apache/airflow/issues/24030#issuecomment-1142742387
This is a great breakdown. We've been putting a lot of effort into standardizing things in the AWS package and this will definitely help. This may be out of scope, or you may want to include it here as well: Another similar standardizing change we have in mind is to convert as many of the hooks as possible into "thin hooks". If a hook method just passes through to a boto call, then there's really no reason to have that method defined and it can/should be removed. For a good example of what we'd like to move towards, see how the `redshift_data` hook and operator interact. There is no reason to redefine `describe_statement()` in the hook because it just passes through to the boto call `describe_statement()`. This makes cleaner code, less maintenance, and more future-proof if the boto API changes. -- 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]
