Skip to content
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: Implement sink for services s3 #2508

Merged
merged 11 commits into from
Jun 23, 2023
Merged

feat: Implement sink for services s3 #2508

merged 11 commits into from
Jun 23, 2023

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Jun 21, 2023

This is a very basic implement of sink. And it's not play well with our existing features like append or write. We will improve it in later PRs.

Close #2441
Part of #2084

Signed-off-by: Xuanwo <[email protected]>
@suyanhanx
Copy link
Member

Where's the content of #2441? 🤔

@Xuanwo
Copy link
Member Author

Xuanwo commented Jun 21, 2023

Where's the content of #2441? thinking

Fixed by changing #2441

Signed-off-by: Xuanwo <[email protected]>
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo Xuanwo requested review from xyjixyjixyji and suyanhanx June 23, 2023 03:00
@Xuanwo Xuanwo merged commit 39e8837 into main Jun 23, 2023
@Xuanwo Xuanwo deleted the implement-sink-for-s3 branch June 23, 2023 05:13
@PsiACE PsiACE mentioned this pull request Jun 27, 2023
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add sink API for Writer
4 participants