ahmarsuhail
ahmarsuhail
Thanks @monthonk! @steveloughran how can the yetus deprecation errors be handled? Also for credential providers, do we want to allow custom V1 credential providers in delegation tokens and otherwise? They...
Thanks @steveloughran, I've suppressed deprecation warnings and used LogExactlyOnce. Yetus is happy now so this is now ready for review. Also had a couple of questions.
@steveloughran have added a new V2Migration class and reverted using fully qualified classnames. Some deprecation warnings now, but they are all for imports. For AWSCredentialProviderList, this is what I was...
Thanks @steveloughran, have made the suggested changes. And yes, data types are changing. [getObject](https://sdk.amazonaws.com/java/api/latest/software/amazon/awssdk/services/s3/S3Client.html#getObject-java.util.function.Consumer-) now returns [ResponseInputStream](https://sdk.amazonaws.com/java/api/latest/software/amazon/awssdk/core/ResponseInputStream.html). In this PR, I'm logging a warning on `getObjectMetadata`, which will now return...
@steveloughran i think you may have viewed changes from a previous commit, may last commit addresses both of those issues: https://github.com/apache/hadoop/pull/4698/commits/3b87159add00a8820e87942cae7462a7fc354390
Thanks @dannycjones, have moved those things to constants. RE using `eu-west-1` and the region logic in general: Not using `us-east-1` as `headBucket()` fails with that region. This is because `us-east-1`...
@steveloughran / @mukund-thakur , I've opened a PR with the first set of changes for the SDK upgrade, could you please take a look?
closing this, these changes will be part of a new PR I'll open once the feature branch is in sync with trunk.
@steveloughran could you please review? I'll then pick up testing for https://github.com/apache/hadoop/pull/4698 (prerequisites for upgrading to V2) against `branch-3.3`.
thanks @steveloughran, have added `EC2ContainerCredentialsProviderWrapper` and `InstanceProfileCredentialsProvider`