Skip to content

Enable instance and custom credential provider for glue#16452

Closed
v-jizhang wants to merge 3 commits intoprestodb:masterfrom
v-jizhang:aws_credential_hive
Closed

Enable instance and custom credential provider for glue#16452
v-jizhang wants to merge 3 commits intoprestodb:masterfrom
v-jizhang:aws_credential_hive

Conversation

@v-jizhang
Copy link
Contributor

@v-jizhang v-jizhang commented Jul 20, 2021

Add config hive.metastore.glue.aws-credentials-provider for glue
Cherry-pick of trinodb/trino#1363,
trinodb/trino#741 and
trinodb/trino#3689

Add config hive.metastore.glue.aws-credentials-provider for glue
credential provider. where value is fully qualified class name of
custom AWS credential provider implementation.

== RELEASE NOTES ==

AWS Changes
* Enable instance and custom credential provider for glue
  Add configuration property
  ``hive.metastore.glue.aws-credentials-provider`` to supply a 
  custom credential provider.

v-jizhang and others added 3 commits July 23, 2021 14:44
Cherry-pick of trinodb/trino#741

DefaultAWSCredentialsProviderChain is frequently used by AWS customers
and it provides access from a documented list of sources. This
especially makes it easier to run Presto on non-EC2 hosts where you
don't have the instance profile. (e.g. Macs, during development). This
change also makes PrestoS3FileSystem to be consistent with the Glue
connector in Presto.

Co-authored-by: Anoop Johnson <anoopj@amazon.com>
Cherry-pick of trinodb/trino#1363

Add a config for glue credential hive.metastore.glue.aws-credentials-provider
where value is fully qualified class name.

Co-authored-by: Li Yu <li.yu.sh0211@gmail.com>
Cherry-pick of trinodb/trino#3689

Add configuration property
  ``hive.metastore.glue.aws-credentials-provider`` to supply a
custom credential provider.

Co-authored-by: Ashhar Hasan <hashhar_dev@outlook.com>
Co-authored-by: Piotr Findeisen <piotr.findeisen@gmail.com>
@v-jizhang v-jizhang force-pushed the aws_credential_hive branch from d5f4013 to a9f5300 Compare July 23, 2021 21:53
@v-jizhang v-jizhang requested a review from aweisberg August 17, 2021 22:31
@stale
Copy link

stale bot commented Mar 2, 2022

This pull request has been automatically marked as stale because it has not had recent activity. If you'd still like this PR merged, please comment on the task, make sure you've addressed reviewer comments, and rebase on the latest master. Thank you for your contributions!

@stale stale bot added the stale label Mar 2, 2022
@stale stale bot closed this Apr 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant