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

Fangmin Lv updated BOOKKEEPER-460:
----------------------------------

    Attachment: BOOKKEEPER-460.patch

@Ivan
Good idea, submit a patch according to your proposal.
                
> LedgerDeleteTest checks wrong place for log file
> ------------------------------------------------
>
>                 Key: BOOKKEEPER-460
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-460
>             Project: Bookkeeper
>          Issue Type: Test
>          Components: bookkeeper-server
>            Reporter: Fangmin Lv
>            Assignee: Fangmin Lv
>            Priority: Trivial
>         Attachments: BOOKKEEPER-460.diff, BOOKKEEPER-460.diff, 
> BOOKKEEPER-460.patch
>
>
> A minor problem, it seems forgot to update LedgerDeleteTest after upgrade 
> bookkeeper filesystem.
> {code:xml}
> // Verify that the first entry log (0.log) has been deleted from all of the 
> Bookie Servers.
> for (File ledgerDirectory : tmpDirs) {
>     for (File f : ledgerDirectory.listFiles()) {
>         assertFalse("Found the entry log file (0.log) that should have been 
> deleted in ledgerDirectory: " 
>             + ledgerDirectory, f.isFile() && f.getName().equals("0.log"));
>     }
> }
> {code} 
> Solution:
> Just reuse the checkLogFiles method in CompactionTest.

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