GitHub user zhaijack opened a pull request: https://github.com/apache/incubator-distributedlog/pull/135
(WIP) DL-2: DistributedLog should work with the official apache bookkeeper This change is to upgrade bookkeeper version to BK 4.5.0. (It is still WIP) - upgrade bookkeeper version to 4.5.0-SNAPSHOT (still waiting a few pull requests to merge apache/bookkeeper#297 apache/bookkeeper#287 - change registerSuccessEvent for StatsLogger to add TimeUnit - use netty4 eventloop - move twitter repository dependencies to proxy related module only. core library will not depend on scala dependency anymore. This change is a collaboration change with @sijie We will provide a performance comparison between 0.4.0 (using Twitter BK) and 0.5.0 (using BK 4.5.) in a separate pull request or email. You can merge this pull request into a Git repository by running: $ git pull https://github.com/zhaijack/incubator-distributedlog bump_dl_version Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-distributedlog/pull/135.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #135 ---- commit 2f25d9fead78d982920fe72720b5470a5cb949f7 Author: Sijie Guo <si...@apache.org> Date: 2017-07-19T23:33:46Z Bump BK version to 4.5.0-SNAPSHOT - registerSuccessEvent and registerFailureEvent needs TimeUnit - change HashedWheelTimer to netty 4 - change channelFactory to eventLoopGroup commit ccd74487da38d0fde64569005b777fea658a0adb Author: Jia Zhai <zhaiji...@gmail.com> Date: 2017-07-27T01:07:52Z DL-2: DistributedLog should work with the official apache bookkeeper version ---- --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---