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

Xiaoyu Yao edited comment on HDFS-10922 at 10/5/16 5:20 PM:
------------------------------------------------------------

bq. Now working on #4 about namenode restart. One question, the patch modified 
both common and hdfs projects, is it OK to use just single 1 patch ?

Good catch, [~cheersyang]. Let's use this ticket for hdfs only changes in #4. 
That should be landed in {{o.a.h.hdfs.TestHDFSTrash}} 

I opened a new ticket HADOOP-13686 for #1-3. I notice the permission test for 
#1 is added for only hdfs. Can you move that to hadoop-common and attach the 
updated patch in HADOOP-13686? This way, we will have better coverage than 
having the test only for hdfs.



was (Author: xyao):
bq. Now working on #4 about namenode restart. One question, the patch modified 
both common and hdfs projects, is it OK to use just single 1 patch ?

Good catch, [~cheersyang]. Let's have a separate ticket for the hdfs changes 
for #4. That should be landed in {{o.a.h.hdfs.TestHDFSTrash}} 

I will move this ticket from hadoop-hdfs to hadoop-common for #1-3 only. I 
notice the permission test for #1 is added for only hdfs. Can you move the test 
to hadoop-common as the behavior should not be specific to hdfs?


> Adding additional unit tests for Trash
> --------------------------------------
>
>                 Key: HDFS-10922
>                 URL: https://issues.apache.org/jira/browse/HDFS-10922
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: test
>            Reporter: Xiaoyu Yao
>            Assignee: Weiwei Yang
>         Attachments: HDFS-10922.02.patch, HDFS-10922.03.patch, 
> HDFS-10922.04.patch, HDFS-10922.05.patch, HDFS-10922.06.patch
>
>
> This ticket is opened to track adding unit tests for Trash.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to