You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "ddupg (via GitHub)" <gi...@apache.org> on 2026/05/18 10:08:39 UTC
[PR] feat(services/tos): implement stat and multipart upload [opendal]
ddupg opened a new pull request, #7552:
URL: https://github.com/apache/opendal/pull/7552
# Which issue does this PR close?
Closes #7152.
# Rationale for this change
Add TOS multipart upload and stat support.
# What changes are included in this PR?
- Implement TOS multipart upload.
- Enable TOS stat capability.
# Are there any user-facing changes?
TOS now supports multipart writes and stat.
# AI Usage Statement
Implemented with OpenAI Codex using GPT-5.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat(services/tos): implement stat and multipart upload [opendal]
Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo commented on code in PR #7552:
URL: https://github.com/apache/opendal/pull/7552#discussion_r3273589628
##########
core/services/tos/src/backend.rs:
##########
@@ -203,7 +203,12 @@ impl Builder for TosBuilder {
list_with_start_after: true,
list_with_recursive: true,
- stat: false,
+ stat: true,
+ stat_with_if_match: true,
+ stat_with_if_none_match: true,
+ stat_with_if_modified_since: true,
+ stat_with_if_unmodified_since: true,
+ stat_with_version: config.enable_versioning,
Review Comment:
This option should be removed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat(services/tos): implement stat and multipart upload [opendal]
Posted by "ddupg (via GitHub)" <gi...@apache.org>.
ddupg commented on code in PR #7552:
URL: https://github.com/apache/opendal/pull/7552#discussion_r3273894770
##########
core/services/tos/src/backend.rs:
##########
@@ -203,7 +203,12 @@ impl Builder for TosBuilder {
list_with_start_after: true,
list_with_recursive: true,
- stat: false,
+ stat: true,
+ stat_with_if_match: true,
+ stat_with_if_none_match: true,
+ stat_with_if_modified_since: true,
+ stat_with_if_unmodified_since: true,
+ stat_with_version: config.enable_versioning,
Review Comment:
Fixed it. Thanks @Xuanwo for reviewing.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat(services/tos): implement stat and multipart upload [opendal]
Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo merged PR #7552:
URL: https://github.com/apache/opendal/pull/7552
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org