On Tue, Oct 18, 2016 at 1:37 PM Josh Elser <[email protected]> wrote:
> Keith Turner wrote: > >> * Saw libthrift in the parent pom.xml, but could not see usages of that > >> > anywhere in the child modules. (not a release issue, just curious) > > > > Its in the dep mgmt section. The reason is its needed to build > > recipes against accumulo 1.8.0. Fluo also uses thrift and depends on > > 0.9.1. If thrift 0.9.1 ends up on path, then MiniAccumulo will hang. > > I can add a comment to the pom. > > > > Ah, ok. I just did a grep and did see any usages of Thrift in the > recipes which made me wonder why it was defined (even if only in > depMgmt). Overriding Fluo's thrift dependency makes sense -- you don't > need to document on my account. > Still nice to document for the next release.
