-
Notifications
You must be signed in to change notification settings - Fork 488
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: impl put_multipart
in object_store
#4793
Conversation
522dfae
to
9990f5f
Compare
@Xuanwo hi, have a moment to review? |
819fce4
to
d4391e3
Compare
d4391e3
to
124788e
Compare
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Hi @Rachelint, thank you for your patience. I attempted to address this issue through OpenDAL's API design but ultimately found it unfeasible, so I implemented a workaround using task notifications instead. The concurrent writing occurs within OpenDAL itself, so the lock here isn't a major concern. |
Thanks for @Xuanwo |
close: #4748