You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bookkeeper.apache.org by Sijie Guo <gu...@gmail.com> on 2012/03/13 09:13:24 UTC

Review Request: BOOKKEEPER-184: CompactionTest failing on Jenkins

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4304/
-----------------------------------------------------------

Review request for bookkeeper.


Summary
-------

he compaction test hanging on Jenkins.

Ivan did some investigation on this issue, and found that it wasn't hanging, it was just taking a really long time. He suggested that it is because it does a lot of I/O, theres a couple of ways we could reduce this.
1. Only create 1 bookie
2. Not inherit from BaseTestCase
3. Reduce the number of entries


This addresses bug BOOKKEEPER-184.
    https://issues.apache.org/jira/browse/BOOKKEEPER-184


Diffs
-----

  bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/CompactionTest.java 34dcc74 
  bookkeeper-server/src/test/java/org/apache/bookkeeper/client/TestReadTimeout.java 8da000e 
  bookkeeper-server/src/test/java/org/apache/bookkeeper/meta/HierarchicalLedgerDeleteTest.java 6fd1c5c 
  bookkeeper-server/src/test/java/org/apache/bookkeeper/meta/LedgerLayoutTest.java 8235ce5 
  bookkeeper-server/src/test/java/org/apache/bookkeeper/meta/TestLedgerManager.java 099eb7c 
  bookkeeper-server/src/test/java/org/apache/bookkeeper/test/BaseTestCase.java 0fe5aba 
  bookkeeper-server/src/test/java/org/apache/bookkeeper/test/BookKeeperClusterTestCase.java PRE-CREATION 
  bookkeeper-server/src/test/java/org/apache/bookkeeper/test/BookieJournalRollingTest.java 17f5da2 
  bookkeeper-server/src/test/java/org/apache/bookkeeper/test/BookieZKExpireTest.java ee5d24e 
  bookkeeper-server/src/test/java/org/apache/bookkeeper/test/IndexCorruptionTest.java 5dcb964 
  bookkeeper-server/src/test/java/org/apache/bookkeeper/test/LedgerDeleteTest.java 5364fdd 

Diff: https://reviews.apache.org/r/4304/diff


Testing
-------


Thanks,

Sijie