You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Daryn Sharp (Commented) (JIRA)" <ji...@apache.org> on 2012/04/02 17:31:25 UTC

[jira] [Commented] (HADOOP-8014) ViewFileSystem does not correctly implement getDefaultBlockSize, getDefaultReplication, getContentSummary

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

Daryn Sharp commented on HADOOP-8014:
-------------------------------------

{{FilterFileSystem}}'s no-path methods should be calling the path variant.  (Sorry I butchered it as {{FileFileSystem}} in an earlier comment).  Pushing this behavior up into {{ChrootFileSystem}} will force any other subclass of {{FilterFileSystem}} that wants to alter the behavior of path handling to implement both methods.  I think putting the behavior in {{FilterFileSystem}} is more future-proof for arbitrarily stackable filesystems.

Otherwise, looks great.
                
> ViewFileSystem does not correctly implement getDefaultBlockSize, getDefaultReplication, getContentSummary
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-8014
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8014
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.23.0, 0.23.2, 0.24.0
>            Reporter: Daryn Sharp
>            Assignee: John George
>         Attachments: HADOOP-8014.patch, HADOOP-8014.patch, HADOOP-8014.patch, HADOOP-8014.patch
>
>
> {{ViewFileSystem}} incorrectly returns the {{FileSystem}} default values for {{getDefaultBlockSize()}} and {{getDefaultReplication()}}.  This causes files to be created with incorrect values.  The problem is that the current apis are insufficient for viewfs because the defaults depend on the underlying mount point.  These methods need counterparts that accept a {{Path}} so viewfs can resolve the mount point for a path.

--
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