[PR #64] [MERGED] Return EIO on failed read #1333

New issue

Closed

opened 2026-03-04 01:53:26 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 01:53:26 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/64
Author: @gaul
Created: 10/4/2014
Status: ✅ Merged
Merged: 11/2/2014
Merged by: @ggtakec

Base: master ← Head: failed-read-eio

📝 Commits (1)

a1ca8b7 Return EIO on failed read

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 src/curl.cpp (+2 -2)

📄 Description

Previously S3fsMultiCurl::MultiRead did not report read errors since
it did not treat failed callback setup as a fatal operation error.
Failed callback setups usually result from exceeding the number of
allowed retries. Previously cp did not report an error during a
network outage but now does:

$ cp ~/s3-path/s3-file .
cp: error reading ‘/home/gaul/s3-path/s3-file’: Input/output error
cp: failed to extend ‘./s3-file’: Input/output error

_{🔄 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/64 **Author:** [@gaul](https://github.com/gaul) **Created:** 10/4/2014 **Status:** ✅ Merged **Merged:** 11/2/2014 **Merged by:** [@ggtakec](https://github.com/ggtakec) **Base:** `master` ← **Head:** `failed-read-eio` --- ### 📝 Commits (1) - [`a1ca8b7`](https://github.com/s3fs-fuse/s3fs-fuse/commit/a1ca8b712401dd242273c138ca1f65d69cfcc605) Return EIO on failed read ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/curl.cpp` (+2 -2) </details> ### 📄 Description Previously S3fsMultiCurl::MultiRead did not report read errors since it did not treat failed callback setup as a fatal operation error. Failed callback setups usually result from exceeding the number of allowed retries. Previously cp did not report an error during a network outage but now does: ``` $ cp ~/s3-path/s3-file . cp: error reading ‘/home/gaul/s3-path/s3-file’: Input/output error cp: failed to extend ‘./s3-file’: Input/output error ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>