[GitHub] sijie commented on issue #1962: Failed to download twitter maven repo dependencies

[GitBox <gi...@apache.org>]

sijie commented on issue #1962: Failed to download twitter maven repo dependencies
URL: https://github.com/apache/bookkeeper/issues/1962#issuecomment-467414467
 
 
   > do we need to support 'twitter-science-provider' in Community builds ?
   
   I think it can still be built as part of `bookkeeper-all` distributions. but I don't think we should build twitter providers in normal processes. because 1) some of the twitter dependencies are only in twitter maven repo, but not at maven center. it causes problems for our CI builds; 2) twitter providers are not very popular as the other two modules - prometheus and codahale. 
   
   so I would suggest exclude them from normal build and only build at part of bookkeeper-all. it will also reduce the build time for most of developers.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services