xingbowu commented on pull request #3659: URL: https://github.com/apache/iceberg/pull/3659#issuecomment-986237320
Hi Jack, Regarding the name style of access key, I agree with you that the best way is to line up them between aws and aliyun module。however, considering legacy reason for user behavior, we decided to keep it as it is. could you check the discussion comments in PR3553 https://github.com/apache/iceberg/pull/3553#discussion_r752046575 feel free to let me know your further idea? it looks good to me to rename `AliyunClientFactories.load` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
