bookkeeper-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject [bookkeeper] branch master updated: Fix test regression due to change in default ledger manager
Date Tue, 13 Feb 2018 14:09:39 GMT
This is an automated email from the ASF dual-hosted git repository.

ivank pushed a commit to branch master
in repository

The following commit(s) were added to refs/heads/master by this push:
     new eae32f6  Fix test regression due to change in default ledger manager
eae32f6 is described below

commit eae32f6b667f0842897f4ca151cc7b7d31ef2e91
Author: Ivan Kelly <>
AuthorDate: Tue Feb 13 15:09:32 2018 +0100

    Fix test regression due to change in default ledger manager
    TestBenchmark#testReadThroughputLatency() regressed due to the change
    in 3ddc5db, where the default ledger manager was changed from flat to
    The fix is to use flat for this test. I've also cleaned up the test a
    little, and added some code to the benchmark to make the test run
    Author: Ivan Kelly <>
    Reviewers: Enrico Olivelli <>, Sijie Guo <>
    This closes #1137 from ivankelly/bench-broke
 .../test/java/org/apache/bookkeeper/benchmark/  | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/bookkeeper-benchmark/src/test/java/org/apache/bookkeeper/benchmark/
index e3b1124..205c4ab 100644
--- a/bookkeeper-benchmark/src/test/java/org/apache/bookkeeper/benchmark/
+++ b/bookkeeper-benchmark/src/test/java/org/apache/bookkeeper/benchmark/
@@ -25,6 +25,7 @@ import;
 import org.apache.bookkeeper.test.BookKeeperClusterTestCase;
 import org.junit.Test;
 import org.junit.Assert;
+import org.junit.Before;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -38,6 +39,13 @@ public class TestBenchmark extends BookKeeperClusterTestCase {
+    @Before
+    public void setUp() throws Exception {
+        baseConf.setLedgerManagerFactoryClassName("org.apache.bookkeeper.meta.FlatLedgerManagerFactory");
+        baseClientConf.setLedgerManagerFactoryClassName("org.apache.bookkeeper.meta.FlatLedgerManagerFactory");
+        super.setUp();
+    }
     public void testThroughputLatency() throws Exception {
         String latencyFile = System.getProperty("test.latency.file", "latencyDump.dat");

To stop receiving notification emails like this one, please contact

View raw message