-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(aws-glue): expose grant API for glue Table #17935
Labels
@aws-cdk/aws-glue
Related to AWS Glue
effort/small
Small work item – less than a day of effort
feature-request
A feature should be added or improved.
p1
Comments
kaizencc
added
feature-request
A feature should be added or improved.
needs-triage
This issue or PR still needs to be triaged.
labels
Dec 9, 2021
kaizencc
added
effort/small
Small work item – less than a day of effort
p1
@aws-cdk/aws-glue
Related to AWS Glue
and removed
@aws-cdk/aws-glue
Related to AWS Glue
needs-triage
This issue or PR still needs to be triaged.
labels
Dec 9, 2021
mergify bot
pushed a commit
that referenced
this issue
Dec 10, 2021
…ns (#17941) It is convention in the CDK to expose the underlying `grant()` API to make it simple for users to grant custom permissions to their resource. In addition, this PR removes 'glue:BatchDeletePartition' from `readPermissions`, which was previously erroneously added. closes #17935 and #15116. BREAKING CHANGE: the grantRead API previously included 'glue:BatchDeletePartition', and now it does not. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
|
TikiTDO
pushed a commit
to TikiTDO/aws-cdk
that referenced
this issue
Feb 21, 2022
…ns (aws#17941) It is convention in the CDK to expose the underlying `grant()` API to make it simple for users to grant custom permissions to their resource. In addition, this PR removes 'glue:BatchDeletePartition' from `readPermissions`, which was previously erroneously added. closes aws#17935 and aws#15116. BREAKING CHANGE: the grantRead API previously included 'glue:BatchDeletePartition', and now it does not. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
@aws-cdk/aws-glue
Related to AWS Glue
effort/small
Small work item – less than a day of effort
feature-request
A feature should be added or improved.
p1
Description
Currently the
table.grant()
API is private:aws-cdk/packages/@aws-cdk/aws-glue/lib/table.ts
Lines 328 to 334 in 42cf186
AFAIK that is not the typical CDK behavior. We usually expose
grant
APIs to unblock users who want to grant custom permissions.Use Case
I want to grant a custom permission to a glue table. Specifically I want to do:
Proposed Solution
Expose the API as
public
.Other information
No response
Acknowledge
The text was updated successfully, but these errors were encountered: