[PR #1128] [MERGED] Fixed MultiRead method to exit after waiting other parts at error #1837

Closed
opened 2026-03-04 02:02:24 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/1128
Author: @ggtakec
Created: 8/12/2019
Status: Merged
Merged: 8/12/2019
Merged by: @gaul

Base: masterHead: fix_multiread


📝 Commits (1)

  • a5f8fe8 Fixed MultiRead method to exit after waiting other parts at error

📊 Changes

1 file changed (+15 additions, -4 deletions)

View changed files

📝 src/curl.cpp (+15 -4)

📄 Description

Relevant Issue (if applicable)

n/a

Details

In multipart processing, when a part reaches the retry count limit due to get some error, an error(EIO) was returned at reaching limit.
However, this is wrong, and the S3fsMultiCurl::MultiRead() method must wait for receiving all part results before returning an error(EIO).


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/s3fs-fuse/s3fs-fuse/pull/1128 **Author:** [@ggtakec](https://github.com/ggtakec) **Created:** 8/12/2019 **Status:** ✅ Merged **Merged:** 8/12/2019 **Merged by:** [@gaul](https://github.com/gaul) **Base:** `master` ← **Head:** `fix_multiread` --- ### 📝 Commits (1) - [`a5f8fe8`](https://github.com/s3fs-fuse/s3fs-fuse/commit/a5f8fe8db92ce97324e715a0e190015e0200d512) Fixed MultiRead method to exit after waiting other parts at error ### 📊 Changes **1 file changed** (+15 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `src/curl.cpp` (+15 -4) </details> ### 📄 Description ### Relevant Issue (if applicable) n/a ### Details In multipart processing, when a part reaches the retry count limit due to get some error, an error(EIO) was returned at reaching limit. However, this is wrong, and the S3fsMultiCurl::MultiRead() method must wait for receiving all part results before returning an error(EIO). --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-04 02:02:24 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/s3fs-fuse#1837
No description provided.