[jira] [Commented] (ACCUMULO-927) Replace HDFS VFS FileSystem objects with the ones from Commons VFS

["Dave Marion (JIRA)" <ji...@apache.org>]

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

Dave Marion commented on ACCUMULO-927:
--------------------------------------

Attached patch that removes the ReadOnlyHdfs* VFS classes and replaces them with the accepted, but not yet released, classes from Commons VFS. I had to change them a little due to interface changes in VFS 2.1 which is not yet released. This change should allow us to remove these classes when Commons VFS 2.1 is released with no code change.

I also removed the test case associated with the ReadOnlyHdfs* classes as the new classes have their own tests in Commons VFS and pass those tests. There was one behavior change with the recent changes to HdfsFileSystem.resolveFile(). It appears that when a file is deleted from HDFS the resolveFile() method still returns true instead of false because the file is still in the local VFS cache. I don't think this will present a problem as the fileChanged event still fires instead of the fileDeleted event. I don't think it will be common that folks delete jars from HDFS without restarting Accumulo as the classes that are loaded from that jar will still be resident in the VM.
                
> Replace HDFS VFS FileSystem objects with the ones from Commons VFS
> ------------------------------------------------------------------
>
>                 Key: ACCUMULO-927
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-927
>             Project: Accumulo
>          Issue Type: Sub-task
>          Components: start
>            Reporter: Dave Marion
>            Assignee: Dave Marion
>             Fix For: 1.5.0
>
>         Attachments: ACCUMULO-927-1.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira