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

Chris Nauroth commented on HDFS-4693:
-------------------------------------

Thanks, Arpit.  It looks like there is still one more indent by 4 instead of 2 
in {{testSecondaryNamenodeError2}}.

I am +1 for the patch though after that very minor nit is corrected.  Thanks 
for the patch!

{code}
@@ -426,6 +436,9 @@ public void testSecondaryNamenodeError2() throws 
IOException {
           replication, seed);
       checkFile(fileSys, file1, replication);
     } finally {
+      if (secondary != null) {
+          secondary.shutdown();
+      }
{code}

                
> Some test cases in TestCheckpoint do not clean up after themselves
> ------------------------------------------------------------------
>
>                 Key: HDFS-4693
>                 URL: https://issues.apache.org/jira/browse/HDFS-4693
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: test
>    Affects Versions: 3.0.0
>            Reporter: Arpit Agarwal
>            Assignee: Arpit Agarwal
>             Fix For: 3.0.0
>
>         Attachments: HDFS-4693.1.patch, HDFS-4693.2.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

Reply via email to