Github user eolivelli commented on the issue:

    https://github.com/apache/bookkeeper/pull/117
  
    @arvindkandhare I have addressed all the issue.
    Basically you forget to add an "include" clause in compat dependencies poms 
and relocate all guava classes.
    As a class (HostAndPort) has been marked as "deprecate" a SuppressWarnings 
annotation needs to be added in the CodahaleMetricsProvider
    
    I have pushed a working branch here
    https://github.com/eolivelli/bookkeeper/tree/guava_version
    
    You can take the last commit
    
https://github.com/eolivelli/bookkeeper/commit/b3d0ae8627c03356e1fd450326a5ef100f5b05dd
    
    I hope that helps
    
    The last thing to do is to ensure that Guava is not used in any "public" 
API (BookKeeper,LedgerHandler, Auth Plugin API, LedgerManager API, 
EnsemblePlacement API...and so on)
    Recently we had to break compatibility because Protobuf was "shaded" and 
used in the Auth Plugin API explicitly
    



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