discivigour commented on code in PR #160:
URL: https://github.com/apache/paimon-rust/pull/160#discussion_r3019533026
##########
crates/paimon/src/io/file_io.rs:
##########
@@ -122,18 +122,21 @@ impl FileIO {
// use normalize_root to make sure it end with `/`.
let list_path = normalize_root(relative_path);
- // Request ContentLength and LastModified so accessing
meta.content_length() / last_modified()
- let entries = op
- .list_with(&list_path)
- .metakey(Metakey::ContentLength | Metakey::LastModified)
- .await
- .context(IoUnexpectedSnafu {
- message: format!("Failed to list files in '{path}'"),
- })?;
+ let entries = op.list_with(&list_path).await.context(IoUnexpectedSnafu
{
+ message: format!("Failed to list files in '{path}'"),
+ })?;
let mut statuses = Vec::new();
-
+ let list_path_normalized = list_path.trim_start_matches('/');
for entry in entries {
+ // opendal list_with includes the root directory itself as the
first entry.
Review Comment:
Because I have updated opendal 0.49 to 0.55 to support
opendal::services::OssConfig's security_token field
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
