Xuanwo

Results 1638 comments of Xuanwo

> I found an issue about Google Drive folder too. I can stat `/assets` but can't stat `/assets/images` or any other path with `/` as separator. Does it cause by...

By the way, are you interested in adding OpenDAL support to `vortex-io`? I'm willing to contribute.

Agreed. By the way, will this change introduce extra cost?

> What kind of cost do you mean? I'm not sure if I understand correctly: `Literal -> Bytes` vs `Literal -> avro::Value -> Bytes`

> I think it's > > `Literal -> RawLiteral -> Bytes` vs `Literal -> avro::Value -> Bytes` I see. Thanks!

I remembered that it's by design that role_arn not loaded from env and config because of the different between AssumeRoleArn and AssumeRoleArnWithWebIdentityToken. Cc @everpcpc do you have comments on this...

> However role_arn from a config file does support normal assumable roles, so if we want to mimic AWS CLI behavior, it needs to be different between env and config...

> But `source_credential` is already supported by reqsign? We only accept `source_credential` in `AssumeRoleLoader`. This doesn't work while user have config like: ```ini [profile marketingadmin] role_arn = arn:aws:iam::123456789012:role/marketingadminrole source_profile =...

> Maybe we could handle like this: [boto/botocore@`7fd4057`/botocore/credentials.py#L1485](https://github.com/boto/botocore/blob/7fd4057baac5298fbf594980f618a57dd51b71ad/botocore/credentials.py#L1485) Agreed. It's reqsign side work, let's migrate to there instead.