-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Destination Redshift: CDK update with least changes #38825
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
dec0dcd
to
829e695
Compare
7118bb2
to
cce8cd3
Compare
cce8cd3
to
e84f601
Compare
e84f601
to
6175305
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.
(this is still safe to merge, right? b/c it's not introducing any of the new classes)
Yeah. there should be no functional changes, just one unused method. |
What
Adapting to last known safe version of CDK before proceeding with rewrite of new interfaces.
This should not have any functional changes.
Review guide
User Impact
Can this PR be safely reverted and rolled back?