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

Steve Loughran commented on HADOOP-15671:
-----------------------------------------

LGTM; looks very like the AWS version

in {{validateCredential}} test
* recommend use LambdaTestUtils.intercept() for the operation-raising-exception 
code.
* if the exception isn't as expected, rather than call fail(), its better to 
rethrow the whole exception.That way the failure details aren't lost

ps HADOOP-14556 is exploring using STS for delegation tokens. Keep that in mind 
as a future extension.

> AliyunOSS: Support Assume Roles in AliyunOSS
> --------------------------------------------
>
>                 Key: HADOOP-15671
>                 URL: https://issues.apache.org/jira/browse/HADOOP-15671
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs/oss
>    Affects Versions: 3.1.0, 2.10.0, 2.9.1, 3.2.0, 3.0.3
>            Reporter: wujinhu
>            Assignee: wujinhu
>            Priority: Major
>         Attachments: HADOOP-15671.001.patch, HADOOP-15671.002.patch, 
> HADOOP-15671.003.patch, HADOOP-15671.004.patch, HADOOP-15671.005.patch, 
> HADOOP-15671.006.patch
>
>
> We will add assume role function in Aliyun OSS.
> For details about assume role and sts token, click the link below:
> [https://www.alibabacloud.com/help/doc-detail/31935.html?spm=a2c5t.11065259.1996646101.searchclickresult.1fad155aKOUvJZ]
>  
> Major Changes:
>  # Stabilise the constructor of CredentialsProvider so that other developers 
> can have their own implementations.
>  #  add assumed role functions for hadoop aliyun module



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to