Skip to content

Enable instance and custom credential provider for glue#16195

Closed
v-jizhang wants to merge 5 commits intoprestodb:masterfrom
v-jizhang:aws_credential_via_hive
Closed

Enable instance and custom credential provider for glue#16195
v-jizhang wants to merge 5 commits intoprestodb:masterfrom
v-jizhang:aws_credential_via_hive

Conversation

@v-jizhang
Copy link
Contributor

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.

Co-authored-by: Li Yu li.yu.sh0211@gmail.com
Co-authored-by: Anoop Johnson anoopj@amazon.com
Co-authored-by: Ashhar Hasan hashhar_dev@outlook.com

== 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.

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.

Co-authored-by: Li Yu <li.yu.sh0211@gmail.com>
Co-authored-by: Anoop Johnson <anoopj@amazon.com>
Co-authored-by: Ashhar Hasan <hashhar_dev@outlook.com>
@v-jizhang v-jizhang requested a review from aweisberg June 2, 2021 17:41
Copy link
Contributor

@aweisberg aweisberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you break apart the three commits so that they map to the original ones? It helps preserve the history and makes them easier to review.

@v-jizhang
Copy link
Contributor Author

Replaced by #16452

@v-jizhang v-jizhang closed this Jul 20, 2021
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.

2 participants