Skip to content

[fix](s3) move s3 providers to fe-common to be accessible for jni reader#35779

Merged
yiguolei merged 1 commit intoapache:branch-2.1from
AshinGau:s3_provider_br21
Jun 3, 2024
Merged

[fix](s3) move s3 providers to fe-common to be accessible for jni reader#35779
yiguolei merged 1 commit intoapache:branch-2.1from
AshinGau:s3_provider_br21

Conversation

@AshinGau
Copy link
Member

@AshinGau AshinGau commented Jun 3, 2024

backport: #35690

PropertyConverter.setS3FsAccess has add customized s3 providers:

public static final List<String> AWS_CREDENTIALS_PROVIDERS = Arrays.asList(
            DataLakeAWSCredentialsProvider.class.getName(),
            TemporaryAWSCredentialsProvider.class.getName(),
            SimpleAWSCredentialsProvider.class.getName(),
            EnvironmentVariableCredentialsProvider.class.getName(),
            IAMInstanceCredentialsProvider.class.getName());

And these providers are set as configuration value of fs.s3a.aws.credentials.provider, which will be used as configuration to build s3 reader in JNI readers. However,
DataLakeAWSCredentialsProvider is in fe-core, that is not dependent by JNI readers, so we have to move s3 providers to `fe-common'.

…der (apache#35690)

`PropertyConverter.setS3FsAccess` has add customized s3 providers:
```
public static final List<String> AWS_CREDENTIALS_PROVIDERS = Arrays.asList(
            DataLakeAWSCredentialsProvider.class.getName(),
            TemporaryAWSCredentialsProvider.class.getName(),
            SimpleAWSCredentialsProvider.class.getName(),
            EnvironmentVariableCredentialsProvider.class.getName(),
            IAMInstanceCredentialsProvider.class.getName());
```
And these providers are set as configuration value of
`fs.s3a.aws.credentials.provider`, which will be used as configuration
to build s3 reader in JNI readers. However,
`DataLakeAWSCredentialsProvider` is in `fe-core`, that is not dependent
by JNI readers, so we have to move s3 providers to `fe-common'.
@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@yiguolei yiguolei merged commit 4f0365e into apache:branch-2.1 Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants