[
https://issues.apache.org/jira/browse/HADOOP-7099?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12980350#action_12980350
]
Todd Lipcon commented on HADOOP-7099:
-------------------------------------
I wouldn't want to duplicate the class inside Hadoop - the goal is to keep the
core small where possible, I think, especially where we're talking about new
interfaces we'll have to support.
Down the road we already have a new parallel set of classes for FS access on
the way - FileContext and the "new" FileSystem. Worth checking out if you
haven't seen them yet.
> 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
> 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.
