Steve Loughran created HADOOP-15348:
---------------------------------------

             Summary: S3A Input Stream bytes read counter isn't getting through 
to StorageStatistics/insturmentation properly
                 Key: HADOOP-15348
                 URL: https://issues.apache.org/jira/browse/HADOOP-15348
             Project: Hadoop Common
          Issue Type: Sub-task
          Components: fs/s3
    Affects Versions: 3.0.0, 3.1.0
            Reporter: Steve Loughran


TL;DR: we should have common storage statistics for bytes read and bytes 
written, and S3A should use them in its instrumentation and have enum names to 
match.

# in the S3AInputStream we call 
{{S3AInstrumentation.StreamStatistics.bytesRead(long)}}, which adds the amount 
to {{bytesRead}}, in a read(), readFully, or forward seek() reading in data
# and in {{S3AInstrumentation.mergeInputStreamStatistics}}, that is pulled into 
streamBytesRead.
# which has a Statistics name of ""stream_bytes_read"
# but that is served up in the Storage statistics as "STREAM_SEEK_BYTES_READ", 
which is the wrong name.
# and there isn't a common name for the counter across other filesystems.

For now: people can use the wrong name in the enum; we may want to think about 
retaining it when adding the correct name. And maybe add a 
@Evolving/@LimitedPrivate scope pair to the enum



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-dev-h...@hadoop.apache.org

Reply via email to