[
https://issues.apache.org/jira/browse/HADOOP-6223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12765285#action_12765285
]
Suresh Srinivas commented on HADOOP-6223:
-----------------------------------------
Comments:
# For all the new classes added, need to add Interface classification, audience
and stability.
# AbstractFileSystem.getMyUri() - remove this and use getUri()
# AbstractFileSystem should be InterfaceAudience.Public right instead of
Private?
This goes only into trunk right?
Nits:
# AbstractFileSystem.java - class comment "interface to for implementors" to
"interface for implementors"
# AbstractFileSystem.java - spell check words, move member variables to the top
of the class
# AbstractFileSystem.java - format method comments no to include extra
indentation.
# DelegateToFileSystem - add comments to describe the class
> New improved FileSystem interface for those implementing new files systems.
> ---------------------------------------------------------------------------
>
> Key: HADOOP-6223
> URL: https://issues.apache.org/jira/browse/HADOOP-6223
> Project: Hadoop Common
> Issue Type: Sub-task
> Reporter: Sanjay Radia
> Assignee: Sanjay Radia
> Attachments: AbstractFileSystem.java, afs1.patch, AFS10.patch,
> AFS11.patch, AFS4.patch, Hdfs.java, Hdfs.java
>
>
> The FileContext API (HADOOP-4952) provides an improved interface for the
> application writer.
> This lets us simplify the FileSystem API since it will no longer need to deal
> with notions of default filesystem [ / ], wd, and config
> defaults for blocksize, replication factor etc. Further it will not need the
> many overloaded methods for create() and open() since
> the FileContext API provides that convenience.
> The FileSystem API can be simplified and can now be restricted to those
> implementing new file systems.
> This jira proposes that we create new file system API, and deprecate
> FileSystem API after a few releases.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
