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

Hudson commented on BOOKKEEPER-460:
-----------------------------------

Integrated in bookkeeper-trunk #819 (See 
[https://builds.apache.org/job/bookkeeper-trunk/819/])
    BOOKKEEPER-460: LedgerDeleteTest checks wrong place for log file (Fangmin 
Lv via ivank) [missing file] (Revision 1413700)

     Result = SUCCESS
ivank : 
Files : 
* 
/zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/util/TestUtils.java

                
> 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