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

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


The following commit(s) were added to refs/heads/master by this push:
     new 175c8de  ISSUE #1339: Cleanup the directories created by 
DbLedgerStorageTest
175c8de is described below

commit 175c8ded8417bb50467218edcce5fc2971f0752e
Author: arunlakshman <arunmuthur...@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 <arunmuthur...@gmail.com>
    
    Reviewers: Enrico Olivelli <eolive...@gmail.com>, Matteo Merli 
<mme...@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 7b44c94..fb0a97a 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
si...@apache.org.

Reply via email to