-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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 S3: Adapt kotlin CDK #36869
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This stack of pull requests is managed by Graphite. Learn more about stacking. |
f10b9f5
to
3eda2a5
Compare
f2a632a
to
0d01d6a
Compare
3eda2a5
to
7a07bb3
Compare
0d01d6a
to
22bf871
Compare
7a07bb3
to
d3ef2dd
Compare
22bf871
to
fcee126
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.
Double checking that moving to the new CDK doesn't implicitly add support for the Async framework or anything like that... that's a future project, yes?
d3ef2dd
to
9f7a953
Compare
fcee126
to
ed6d0dc
Compare
@evantahler Yes. well its not even an straightforward feat, our "Async" framework only works with the assumption of CSV Buffers atm. Just making it move ahead with main, so we can continue with required fixes for Parquet/Avro etc. |
9f7a953
to
49efc12
Compare
ed6d0dc
to
c272fdd
Compare
49efc12
to
38192b8
Compare
c272fdd
to
6d82116
Compare
38192b8
to
a3a67e2
Compare
6d82116
to
7860c19
Compare
No description provided.