[GitHub] ctubbsii opened a new pull request #979: Make libthrift dependency "provided"

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

ctubbsii opened a new pull request #979: Make libthrift dependency "provided"
URL: https://github.com/apache/fluo/pull/979
 
 
   Ensure that libthrift does not get included in the set of transitive
   dependencies coming from fluo-core, so that it comes from Accumulo instead.
   
   The specific version of libthrift required for a particular Fluo application at
   runtime must be compatible with both the libthrift version required by the Fluo
   Oracle service and also the Accumulo client code. Since Fluo applications are
   both an Accumulo client and a Fluo Oracle client, and it does not do class path
   isolation between the different client dependencies for each, the version of
   libthrift used must be compatible with both.
   
   Luckily, the meager use of of thrift for Fluo's Oracle means that its client is
   compatible with many versions of thrift, so Fluo applications should use the
   version which is most compatible with Accumulo. Marking this dependency as
   "provided" in fluo-core will make Fluo applications depending on fluo-core
   transitively use the version coming from their Accumulo dependency instead of
   their fluo-core dependency.

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