hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Owen O'Malley (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HADOOP-7099) Make RawLocalFileSystem more friendly to sub-classing
Date Wed, 12 Jan 2011 19:16:48 GMT

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

Owen O'Malley commented on HADOOP-7099:

Actually, this will help fix another annoyance where the RawLocal and Local FileSystem both
use "file:" as their protocol. We probably should use "file:" for raw and "filecrc:" for checksummed.

> Make RawLocalFileSystem more friendly to sub-classing
> -----------------------------------------------------
>                 Key: HADOOP-7099
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7099
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>    Affects Versions: 0.23.0
>            Reporter: Noah Watkins
>            Priority: Minor
>             Fix For: 0.23.0
>         Attachments: HADOOP-7099.patch
> This patch does 3 things that makes sub-classing RawLocalFileSystem easier.
> First, it adds a constructor that allows a sub-class to avoid calling getInitialWorkingDirectory().
This is important because if a sub-class has an initially null uri (prior to initialize()
being called), then getInitialWorkingDirectory() will cause a NullPointerException when it
tries to work with the FS's uri.
> Second, allows subclasses to modify the working directory.
> The third thing this patch does is change loadPermissions to not pass the URI object
to the File(URI) constructor, but rather pass the string representation of the path. This
is important because URI's that are not using the "file" scheme will cause the File(URI) constructor
to throw an exception.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message