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

Kai Zheng commented on HADOOP-14397:
------------------------------------

[~ste...@apache.org],

Should the following from your comments elsewhere be handled here? Thanks!
{quote}
2. There are only tests for HDFS and local, neither of them perfect. Proposed: 
move to AbstractContractCreateTest, test for all filesystems, fix tests and FS 
where appropriate.
3. Add more tests to generate the failure conditions implied by the updated 
filesystem spec. Eg. create over a an existing file, create over a directory, 
create with negative buffer size, negative block size, empty dest path, etc, 
etc. 
This will clarify when precondition checks are made, as well as whether. For 
example: should newFSDataOutputStreamBuilder() validate the path immediately?
{quote}

> Pull up the builder pattern to FileSystem and add AbstractContractCreateTest 
> for it
> -----------------------------------------------------------------------------------
>
>                 Key: HADOOP-14397
>                 URL: https://issues.apache.org/jira/browse/HADOOP-14397
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: common, fs, hdfs-client
>    Affects Versions: 2.9.0
>            Reporter: Lei (Eddy) Xu
>
> After reach the stability of the Builder APIs, we should promote the API from 
> {{DistributedFileSystem}} to {{FileSystem}}, and add necessary contract tests 
> to cover the API for all file systems.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to