You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-issues@hadoop.apache.org by "Tsz Wo (Nicholas), SZE (JIRA)" <ji...@apache.org> on 2012/06/04 23:43:23 UTC

[jira] [Created] (HDFS-3503) Move LengthInputStream and PositionTrackingInputStream to common

Tsz Wo (Nicholas), SZE created HDFS-3503:
--------------------------------------------

             Summary: Move LengthInputStream and PositionTrackingInputStream to common
                 Key: HDFS-3503
                 URL: https://issues.apache.org/jira/browse/HDFS-3503
             Project: Hadoop HDFS
          Issue Type: Improvement
            Reporter: Tsz Wo (Nicholas), SZE
            Assignee: Tsz Wo (Nicholas), SZE


We have LengthInputStream in org.apache.hadoop.hdfs.server.datanode.fsdataset and PositionTrackingInputStream in org.apache.hadoop.hdfs.server.namenode.FSEditLogLoader.  These two classes are generally useful.  Let's move them to org.apache.hadoop.io.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (HDFS-3503) Move LengthInputStream and PositionTrackingInputStream to common

Posted by "Tsz Wo (Nicholas), SZE (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HDFS-3503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13288964#comment-13288964 ] 

Tsz Wo (Nicholas), SZE commented on HDFS-3503:
----------------------------------------------

Hi Eli, there are multiple places having position() and length() methods.  So I think these two implementations are generally useful.
                
> Move LengthInputStream and PositionTrackingInputStream to common
> ----------------------------------------------------------------
>
>                 Key: HDFS-3503
>                 URL: https://issues.apache.org/jira/browse/HDFS-3503
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>            Reporter: Tsz Wo (Nicholas), SZE
>            Assignee: Tsz Wo (Nicholas), SZE
>
> We have LengthInputStream in org.apache.hadoop.hdfs.server.datanode.fsdataset and PositionTrackingInputStream in org.apache.hadoop.hdfs.server.namenode.FSEditLogLoader.  These two classes are generally useful.  Let's move them to org.apache.hadoop.io.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (HDFS-3503) Move LengthInputStream and PositionTrackingInputStream to common

Posted by "Colin Patrick McCabe (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HDFS-3503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13288974#comment-13288974 ] 

Colin Patrick McCabe commented on HDFS-3503:
--------------------------------------------

Seems reasonable.  If you move these, remember to move StreamLimiter as well.
                
> Move LengthInputStream and PositionTrackingInputStream to common
> ----------------------------------------------------------------
>
>                 Key: HDFS-3503
>                 URL: https://issues.apache.org/jira/browse/HDFS-3503
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>            Reporter: Tsz Wo (Nicholas), SZE
>            Assignee: Tsz Wo (Nicholas), SZE
>
> We have LengthInputStream in org.apache.hadoop.hdfs.server.datanode.fsdataset and PositionTrackingInputStream in org.apache.hadoop.hdfs.server.namenode.FSEditLogLoader.  These two classes are generally useful.  Let's move them to org.apache.hadoop.io.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (HDFS-3503) Move LengthInputStream and PositionTrackingInputStream to common

Posted by "Eli Collins (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HDFS-3503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13288936#comment-13288936 ] 

Eli Collins commented on HDFS-3503:
-----------------------------------

Hey Nicholas,
What's the particular motivation in this case?
                
> Move LengthInputStream and PositionTrackingInputStream to common
> ----------------------------------------------------------------
>
>                 Key: HDFS-3503
>                 URL: https://issues.apache.org/jira/browse/HDFS-3503
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>            Reporter: Tsz Wo (Nicholas), SZE
>            Assignee: Tsz Wo (Nicholas), SZE
>
> We have LengthInputStream in org.apache.hadoop.hdfs.server.datanode.fsdataset and PositionTrackingInputStream in org.apache.hadoop.hdfs.server.namenode.FSEditLogLoader.  These two classes are generally useful.  Let's move them to org.apache.hadoop.io.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira