----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8035/ -----------------------------------------------------------
(Updated Dec. 6, 2012, 9:36 a.m.) Review request for bookkeeper. Changes ------- Change follow Ivan and Fangmin's comments. Test Total time that without MSLedgerManagerFactory: 15:17.931s Test Total time that involve MSLedgerManagerFactory: 21:24.679s Description ------- Implement a MetaStore based ledger manager for bookkeeper client. This addresses bug BOOKKEEPER-205. https://issues.apache.org/jira/browse/BOOKKEEPER-205 Diffs (updated) ----- bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BKException.java 749fdb9 bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/AbstractConfiguration.java 536403a bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/MSLedgerManagerFactory.java PRE-CREATION bookkeeper-server/src/test/java/org/apache/bookkeeper/client/BookieRecoveryTest.java 9d38dd0 bookkeeper-server/src/test/java/org/apache/bookkeeper/metastore/TestMetaStore.java 35151df bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/BookieLedgerIndexTest.java ead3494 bookkeeper-server/src/test/java/org/apache/bookkeeper/test/BookKeeperClusterTestCase.java 12e4afc bookkeeper-server/src/test/java/org/apache/bookkeeper/test/MultiLedgerManagerMultiDigestTestCase.java 9630d46 bookkeeper-server/src/test/java/org/apache/bookkeeper/test/MultiLedgerManagerTestCase.java dd3450c Diff: https://reviews.apache.org/r/8035/diff/ Testing ------- Thanks, Jiannan Wang