[HADOOP-16155] S3AInputStream read(bytes[]) to not retry on read failure: pass action up

• Type: Sub-task
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 3.2.0
• Component/s: fs/s3
• Labels: pull-request-available
• Assignee: Ahmar Suhail
• Reporter: Steve Loughran
• Created: 28/Feb/19 18:59
• Updated: 04/Oct/22 16:23

Description

The S3AInputStream reacts to read(byte[]) failure by reopening the stream, just as for the single byte read(). We shouldn't need to do that. Instead just close the stream, return 0 and let the caller decided what to do.

why so?

1. its in the contract of InputStream.read(bytes[]),
2. readFully() can handle the 0 in its loop
3. other apps can decided what to do.

Issue Links

• links to: GitHub Pull Request #3905