[
https://issues.apache.org/jira/browse/HADOOP-7099?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Noah Watkins updated HADOOP-7099:
---------------------------------
Description:
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.
was:
This patch does 2 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.
The second 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.
> 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
>
>
> 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.
