Details
-
Sub-task
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
HADOOP-12756
-
None
-
Reviewed
Description
looking at the AliyunOss integration codes, some findings:
1. TemporaryAliyunCredentialsProvider could be better named;
2. TemporaryAliyunCredentialsProvider shared many codes with AliyunOSSUtils#getCredentialsProvider, and the dup can be resolved;
3. AliyunOSSUtils#getPassword is rather confusing, as used to get other things than password.