Thank you @singhpk234 @flyrain for the review.
As suggested, I have updated the PR description to include the original PR and author. Please let me know if it looks good.

I would wait at least for couple of days for @ebelgasmi12 to respond. Maybe, we can cherry pick commits and add it to original PR, so that original PR can be merged

@flyrain @singhpk234 @jackye1995 @RussellSpitzer
If the changes looks good, would it be possible merge the changes? It would great if this can make it to 1.7 release FYI @ebelgasmi12

Wondering if the aws client can allow missing the client.region config now

@flyrain what do you mean by this? I think that config has been optional? And it will try to resolve the region using the default region chain https://docs.aws.amazon.com/sdk-for-java/latest/developer-guide/region-selection.html#automatically-determine-the-aws-region-from-the-environment

@jackye1995, I meant the AwsClientProperties client.region. I believe it is still mandatory, no? To be clear, it is not related to this PR. I was wondering it since s3 basically can allow client to not set region now.

Thanks @munendrasn for the PR. Thanks all for the review.

I meant the AwsClientProperties client.region. I believe it is still mandatory, no?

No I don't think it is mandatory, I never set this in my sessions at least. If you find it otherwise, we should probably fix it.

Thanks @ebelgasmi12 for the original changes, Thank you all for the review