Skip to content

Commit

Permalink
fix(timestream): magneticStoreWriteProperties and retentionProperties…
Browse files Browse the repository at this point in the history
… not working as Json (aws#23425)

Fixes aws#23404 


----

### All Submissions:

* [x] Have you followed the guidelines in our [Contributing guide?](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md)

### Adding new Construct Runtime Dependencies:

* [ ] This PR adds new construct runtime dependencies following the process described [here](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md/#adding-construct-runtime-dependencies)

### New Features

* [ ] Have you added the new feature to an [integration test](https://github.com/aws/aws-cdk/blob/main/INTEGRATION_TESTS.md)?
	* [ ] Did you use `yarn integ` to deploy the infrastructure and generate the snapshot (i.e. `yarn integ` without `--dry-run`)?

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
madeline-k authored and Brennan Ho committed Feb 22, 2023
1 parent 3f48aab commit bfc39e6
Showing 1 changed file with 29 additions and 0 deletions.
@@ -0,0 +1,29 @@
{
"ResourceTypes": {
"AWS::Timestream::Table": {
"patch": {
"description": "These types were once typed as Json, and adding types now is a breaking change. Keep them as Json forever.",
"operations": [
{
"op": "remove",
"path": "/Properties/MagneticStoreWriteProperties/Type"
},
{
"op": "add",
"path": "/Properties/MagneticStoreWriteProperties/PrimitiveType",
"value": "Json"
},
{
"op": "remove",
"path": "/Properties/RetentionProperties/Type"
},
{
"op": "add",
"path": "/Properties/RetentionProperties/PrimitiveType",
"value": "Json"
}
]
}
}
}
}

0 comments on commit bfc39e6

Please sign in to comment.