You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by si...@apache.org on 2018/06/04 20:41:21 UTC

[bookkeeper] 03/03: ISSUE #1339: Cleanup the directories created by DbLedgerStorageTest

This is an automated email from the ASF dual-hosted git repository.

sijie pushed a commit to branch branch-4.7
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git

commit a30d2fe7a0e7c4ddf49a0c32ecafc02969800d29
Author: arunlakshman <ar...@gmail.com>
AuthorDate: Sat May 12 11:13:29 2018 +0800

    ISSUE #1339: Cleanup the directories created by DbLedgerStorageTest
    
    Descriptions of the changes in this PR:
    
      Cleaned up the directories `dir1` and `dir2` left behind by the test `DbLedgerStorageTest`
    
    Master Issue: #1339
    
    Author: arunlakshman <ar...@gmail.com>
    
    Reviewers: Enrico Olivelli <eo...@gmail.com>, Matteo Merli <mm...@apache.org>, Sijie Guo <si...@apache.org>
    
    This closes #1398 from arunlakshman/issue1339, closes #1339
---
 .../org/apache/bookkeeper/bookie/storage/ldb/DbLedgerStorageTest.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/storage/ldb/DbLedgerStorageTest.java b/bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/storage/ldb/DbLedgerStorageTest.java
index 67c69fe..f3a5b69 100644
--- a/bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/storage/ldb/DbLedgerStorageTest.java
+++ b/bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/storage/ldb/DbLedgerStorageTest.java
@@ -241,10 +241,12 @@ public class DbLedgerStorageTest {
     @Test
     public void doubleDirectory() throws Exception {
         int gcWaitTime = 1000;
+        File firstDir = new File(tmpDir, "dir1");
+        File secondDir = new File(tmpDir, "dir2");
         ServerConfiguration conf = TestBKConfiguration.newServerConfiguration();
         conf.setGcWaitTime(gcWaitTime);
         conf.setLedgerStorageClass(DbLedgerStorage.class.getName());
-        conf.setLedgerDirNames(new String[] { "dir1", "dir2" });
+        conf.setLedgerDirNames(new String[] { firstDir.getCanonicalPath(), secondDir.getCanonicalPath() });
 
         // Should not fail
         Bookie bookie = new Bookie(conf);

-- 
To stop receiving notification emails like this one, please contact
sijie@apache.org.