Fix S3Hook get_key method by removing load extra args

[hussein-awala]

S3 Object.load() doesn't accept any argument, it just returns the metadata of the object. However, the download method should provide the transfer configuration.

[Taragolis]

Actually it does accept, it just not well documented, maybe because boto3.resoure abandoned, at least how it look from my perspective as a regular user of boto3.

And example how to check that it accept you could have a look into the discussion in PR which start propagate this extra values to this method: #35037 (comment)

[hussein-awala]

Actually it does accept, it just not well documented, maybe because boto3.resoure abandoned, at least how it look from my perspective as a regular user of boto3.

And example how to check that it accept you could have a look into the discussion in PR which start propagate this extra values to this method: #35037 (comment)

Hummm interesting 🤔

I will check if these configurations are used where we call other methods on the object.

[Taragolis]

Long term solution would be, get rid of (deprecate) boto3.resource usage in favor of boto3.client:

1. Looks like this objects frozen in a time and do not support all of the features which provided by a client
2. boto3.resource, especially in S3Hook implementation, consume quite a few amount of memory. Reduce memory usage in S3Hook #35449

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.