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

Sanjay Radia commented on HADOOP-4044:
--------------------------------------

Went though the patch. Looks good.
Feedback:
 1) Why does createSymLink create the group name as "none" rather than pass a 
null parameter to use the
group name from the parent dir  similar to the create and mkdir operations.

2) rename and createSymlink both take two path parameters - either of the paths 
can traverse a symlink.
   createSymlink does not handle this while rename does.
   An alternate way to handle this (compared to how rename does) is to have the 
FsResult return a flag saying which of the two paths cross a symlink.

3) getFileInode throws UnresolvedPathException. However, 
LeaseManager:findPath() was changed to
throw IOException (it threw no exceptions before). findPath should be declared 
to throw only the UnresolvedPathException.

4) There is a test to check that symlinks are correctly persisted in 
fsimage/editslogs. 
Please add TestSymlinks that tests for symlinks for each of FileSystem methods 
that take one or more path names. For those that take multiple path names, the 
test should test for symlinks in either parameter.



> Create symbolic links in HDFS
> -----------------------------
>
>                 Key: HADOOP-4044
>                 URL: https://issues.apache.org/jira/browse/HADOOP-4044
>             Project: Hadoop Core
>          Issue Type: New Feature
>          Components: dfs
>            Reporter: dhruba borthakur
>            Assignee: dhruba borthakur
>         Attachments: 4044_20081030spi.java, HADOOP-4044-strawman.patch, 
> symLink1.patch, symLink1.patch, symLink11.patch, symLink4.patch, 
> symLink5.patch, symLink6.patch, symLink8.patch, symLink9.patch
>
>
> HDFS should support symbolic links. A symbolic link is a special type of file 
> that contains a reference to another file or directory in the form of an 
> absolute or relative path and that affects pathname resolution. Programs 
> which read or write to files named by a symbolic link will behave as if 
> operating directly on the target file. However, archiving utilities can 
> handle symbolic links specially and manipulate them directly.

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

Reply via email to