[jira] [Commented] (HADOOP-18189) S3PrefetchingInputStream to support status probes when closed

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HADOOP-18189?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17618198#comment-17618198 ] 

ASF GitHub Bot commented on HADOOP-18189:
-----------------------------------------

virajjasani commented on PR #5036:
URL: https://github.com/apache/hadoop/pull/5036#issuecomment-1279889136

   Tested against `us-west-2` with:
   
   `mvn -Dparallel-tests -DtestsThreadCount=8 -Dscale clean verify`
   
   `mvn -Dparallel-tests -DtestsThreadCount=8 -Dprefetch -Dscale clean verify`
   




> S3PrefetchingInputStream to support status probes when closed
> -------------------------------------------------------------
>
>                 Key: HADOOP-18189
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18189
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>    Affects Versions: 3.4.0
>            Reporter: Steve Loughran
>            Assignee: Viraj Jasani
>            Priority: Minor
>
> S3PrefetchingInputStream is a bit over aggressive on raising exceptions/downgrading responses after a stream is closed
> * MUST: getPos() to return last read location, or a least 0 (maybe we should add this to filesystem spec)
> * MUST: getIOStatistics(). critical for collecting stats in processes
> * MUST: seekToNewSource() (it's a no op anyway)
> * MAY S3AInputStreamStatistics and getStreamStatistics() though that is only in used in testing...ioStatistics have given us a stable stats api. we may want to tag the method @VisibleForTesting to discourage use.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org