Mailing-List: contact common-issues-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: common-issues@hadoop.apache.org
Message-ID: <939045250.1255473211355.JavaMail.jira@brutus>
Date: Tue, 13 Oct 2009 15:33:31 -0700 (PDT)
From: "Suresh Srinivas (JIRA)" <jira@apache.org>
To: common-issues@hadoop.apache.org
Subject: [jira] Commented: (HADOOP-6223) New improved FileSystem interface
 for those implementing new files systems.
In-Reply-To: <9646781.1251678812835.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ 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.