[ 
https://issues.apache.org/jira/browse/HADOOP-18382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17578457#comment-17578457
 ] 

ASF GitHub Bot commented on HADOOP-18382:
-----------------------------------------

ahmarsuhail commented on code in PR #4698:
URL: https://github.com/apache/hadoop/pull/4698#discussion_r943437649


##########
hadoop-tools/hadoop-aws/src/site/markdown/tools/hadoop-aws/index.md:
##########
@@ -233,7 +234,10 @@ needs the credentials needed to interact with buckets.
 
 The client supports multiple authentication mechanisms and can be configured 
as to
 which mechanisms to use, and their order of use. Custom implementations
-of `com.amazonaws.auth.AWSCredentialsProvider` may also be used.
+of `com.amazonaws.auth.AWSCredentialsProvider` may also be used. 
+However, with the upcoming upgrade to AWS Java SDK V2, these classes will need 
to be 

Review Comment:
   Does this look ok? Just to confirm:
   
   - We are deprecating all classes that implement 
`com.amazonaws.auth.AWSCredentialsProvider`. 
   - We will update internal hadoop credential provider classes to implement 
V2's credential provider. Class names will stay the same, so no changes need to 
be made to configs.
   - We will also update delegation token binding classes and credential 
providers to V2, this is a breaking change but this is ok. 
   - We're using a shim for credential providers, so custom cred providers 
which implement V1's Credential provider will still work, but we want to 
encourage people to update these. 





> Upgrade AWS SDK to V2 - Prerequisites 
> --------------------------------------
>
>                 Key: HADOOP-18382
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18382
>             Project: Hadoop Common
>          Issue Type: Sub-task
>            Reporter: Ahmar Suhail
>            Assignee: Ahmar Suhail
>            Priority: Minor
>              Labels: pull-request-available
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> We want to update the AWS SDK to V2, before we do this we should warn on 
> things that will no longer supported. The following changes should be made:
>  
>  * 
> [getAmazonS3Client()|https://github.com/apache/hadoop/blob/221eb2d68d5b52e4394fd36cb30d5ee9ffeea7f0/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java#L1174]
>  - Warn that this method will be removed 
>  * 
> [initCustomSigners()|https://github.com/apache/hadoop/blob/03cfc852791c14fad39db4e5b14104a276c08e59/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/auth/SignerManager.java#L65]
>  - Warn that the interface is changing, any custom signers will need to be 
> updated
>  * 
> [bindAWSClient|https://github.com/apache/hadoop/blob/trunk/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AFileSystem.java#L840]
>  - If DT is enabled, warn that credential providers interface is changing, 
> any custom cred providers used in binding classes will need to be updated
>  *  
> [buildAWSProviderList|https://github.com/apache/hadoop/blob/trunk/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3AUtils.java#L618]
>  - if any SDK V1 cred providers are in this list, warn that these will be 
> removed
>  * 
> [S3ClientFactory|https://github.com/apache/hadoop/blob/221eb2d68d5b52e4394fd36cb30d5ee9ffeea7f0/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/S3ClientFactory.java]
>  - Update javadocs to say this interface will be replaced by a V2 client 
> factory, mark interface deprecated?



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to