-
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
feat(dynamodb): enable contributor insights for global secondary index #30560
base: main
Are you sure you want to change the base?
Conversation
This PR cannot be merged because it has conflicts. Please resolve them. The PR will be considered stale and closed if it remains in an unmergeable state. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your PR! I've added some small comments.
...esting/framework-integ/test/aws-dynamodb/test/integ.dynamodb.contirubtor-insights-for-gsi.ts
Outdated
Show resolved
Hide resolved
import { IntegTest } from '@aws-cdk/integ-tests-alpha'; | ||
|
||
// CDK parameters | ||
const STACK_NAME = 'aws-cdk-dynamodb-contributor-insights-for-gis'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the meaning of gis
? If it means global secondary index, it is better to use gsi
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As you pointed out, it was a mistake.
I have corrected it.
partitionKey: GSI_PARTITION_KEY, | ||
}); | ||
|
||
new IntegTest(app, 'aws-cdk-dynamodb-contributor-insights-for-gis-test', { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just like above, I'm only concerned about gis
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above.
Co-authored-by: Kazuho Cryer-Shinozuka <malaysia.cryer@gmail.com>
…st/integ.dynamodb.contirubtor-insights-for-gsi.ts Co-authored-by: Kazuho Cryer-Shinozuka <malaysia.cryer@gmail.com>
072b75e
to
716c60f
Compare
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
@badmintoncryer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Issue # (if applicable)
Closes #15671 .
Reason for this change
To enable contributor insights for a specific global secondary index.
This feature is supported in CFn.
Description of changes
Add
contributorInsightsEnabled
property to theGlobalSecondaryIndexProps
inTable
.In AWS::DynamoDB::GlobalTable GlobalSecondaryIndex,
ContributorInsightsSpecification
does not exist.So I didn't change
TableV2
.Description of how you validated changes
Add unit test and integ tests.
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license