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

Shradha Revankar commented on HDFS-8885:
----------------------------------------

Hi [~cnauroth],
Thank you for reviewing this, {{getInputStream()}} is called in the constructor 
of {{ByteRangeInputStream}}, so 1 and 2 should not occur. Do you still think 
this should be called again in {{avaliable}}?
3. Agree, will add this check.
4.I will add testcases, sorry about not doing it in the first patch.

> ByteRangeInputStream used in webhdfs does not override available()
> ------------------------------------------------------------------
>
>                 Key: HDFS-8885
>                 URL: https://issues.apache.org/jira/browse/HDFS-8885
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: webhdfs
>    Affects Versions: 2.7.1
>            Reporter: Shradha Revankar
>            Assignee: Shradha Revankar
>            Priority: Minor
>         Attachments: HDFS-8885.000.patch, HDFS-8885.001.patch
>
>
> ByteRangeInputStream has to override the method {{available()}} , the super 
> class implementation returns 0. Clients using the method {{available()}} to 
> rely on available bytes information will end up with errors during reads if 
> WebhdfsFileSystem is used. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to