Github user arvindkandhare commented on the issue:

    https://github.com/apache/incubator-distributedlog/pull/123
  
    While the compile passes, the tests actually try to create a server 
process. This tries to start bookkeeper server, which has dependencies on Guava 
again. This means that this PR needs changes to the Bookkeeper pulled from: 
https://github.com/twitter/bookkeeper/. 
    As Guava has moved and changed the API quiet a bit, this is an involved 
changed. Another interesting point is that Apache bookkeeper uses even older 
version of Guava (13.0 in their master branch). So this PR should not go in 
before the Twitter bookkeeper moves towards this version.
    
    Do share your views on this.



---
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.
---

Reply via email to