-
Notifications
You must be signed in to change notification settings - Fork 15
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(aws-redshiftserverless): support Namespace and Workgroup L2 construct #43
Conversation
9e7dea4
to
35629f6
Compare
7aa155f
to
3b71877
Compare
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.
I'm only halfway through and will see the rest later.
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
Co-authored-by: Kenta Goto (k.goto) <[email protected]> Signed-off-by: mazyu36 <[email protected]>
@go-to-k |
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.
Appreciate the changes! This PR is very good. I approve it.
CI with build also passed. Can we merge this PR? |
Thanks! |
Issue # (if applicable)
Closes #44 .
Reason for this change
To support Redshift Serverless Namespace and Workgroup L2 Construct.
Description of changes
Add
Namespace
andWorkgroup
class.In general, I am implementing properties based on those supported by
aws-cdk-lib
v2.120.0.backlog: #45
Description of how you validated changes
Add unit tests and integ test.
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license