Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

Fix TableV2 construct with streaming enabled#11012

Merged
giograno merged 2 commits intomasterfrom
ddbstream-v2
Jun 14, 2024
Merged

Fix TableV2 construct with streaming enabled#11012
giograno merged 2 commits intomasterfrom
ddbstream-v2

Conversation

@giograno
Copy link
Copy Markdown
Member

Motivation

LocalStack is not able to deploy a TableV2 construct with StreamSpecification enabled.
A sample was provided by a user and is reflected in the implemented test.
Specifying the stream view type in the new construct should be enough to enable the stream (while LocalStack was failing with a missing required field).

Changes

  • Improve provider for AWS::DynamoDB::GlobalTable to account for StreamSpecification;
  • Write a validated CDK test.

@giograno giograno added aws:dynamodb Amazon DynamoDB semver: patch Non-breaking changes which can be included in patch releases labels Jun 12, 2024
@giograno giograno self-assigned this Jun 12, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 12, 2024

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 35m 37s ⏱️ -53s
3 081 tests +1  2 735 ✅ +1  346 💤 ±0  0 ❌ ±0 
3 083 runs  +1  2 735 ✅ +1  348 💤 ±0  0 ❌ ±0 

Results for commit f3ff2ef. ± Comparison against base commit 1823b5d.

♻️ This comment has been updated with latest results.

@giograno giograno marked this pull request as ready for review June 13, 2024 00:06
@giograno giograno merged commit 14dae3f into master Jun 14, 2024
@giograno giograno deleted the ddbstream-v2 branch June 14, 2024 13:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

aws:dynamodb Amazon DynamoDB semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants