[ 
https://issues.apache.org/jira/browse/HADOOP-6017?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Raghu Angadi updated HADOOP-6017:
---------------------------------

    Attachment: HADOOP-6017-branch-18.patch

+1. The patch looks good.

I am attaching the patch 0.18 branch. 

There are more changes required for TestRenameWhileOpen.java since the rename 
test that the new test is part of was disabled in 0.18. This patch enables 
parts of the test.


> NameNode and SecondaryNameNode fail to restart because of abnormal filenames.
> -----------------------------------------------------------------------------
>
>                 Key: HADOOP-6017
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6017
>             Project: Hadoop Core
>          Issue Type: Bug
>    Affects Versions: 0.18.3
>            Reporter: Raghu Angadi
>            Assignee: Tsz Wo (Nicholas), SZE
>            Priority: Blocker
>             Fix For: 0.18.4, 0.19.2, 0.20.1, 0.21.0
>
>         Attachments: 6017_20090611.patch, 6017_20090611b.patch, 
> HADOOP-6017-branch-18.patch
>
>
> SecondaryNameNode (and NameNode) fail to load the edits log.  I will include 
> stack trace in next comment.
> This is traced to the fact that LeaseManager uses {{String.relaceFirst()}} to 
> replace front of a sting with another string. Unfortunately 
> {{replaceFirst()}} uses regex, though the first argument is {{quoted}} by the 
> code, the second argument is not. (the second arg is not really treated as 
> regex but still gets processed for back references (as in '{{sed 
> s/first/second/g}}')
> As Nicholas suggested, it is just simpler to use {{substring()}} to replace 
> part of the string.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to