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 
index e3b1124..205c4ab 100644
@@ -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 {
+        super.setUp();
+    }
     public void testThroughputLatency() throws Exception {
         String latencyFile = System.getProperty("test.latency.file", 

To stop receiving notification emails like this one, please contact

Reply via email to