For info we decided to took the approach 2 Chris will open a PR to do so and we will get it fixed ASAP. Thanks Alex for reporting and sorry for the inconvenience.
On Tue, Feb 18, 2020 at 9:36 PM Ismaël Mejía <[email protected]> wrote: > Mmm that one could be on me, I tested it locally and merged the PR today, > it worked flawlessly in my machine, but now I realize that I have thrift > installed locally because it is a requirement when you hack on parquet-mr > code. > > So two workarounds possible: > > 1. We install Thrift 0.12 in all the Jenkins workers + update instructions > so everyone does. > 2. We disable the Thrift compiler step and use a generated class and > upgrade that every time. > > I am away from my work computer so can someone help me please to unblock > this otherwise I will take care tomorrow. > > > > On Tue, Feb 18, 2020 at 7:43 PM Alex Van Boxel <[email protected]> wrote: > >> I think the Thrift PR broke master. I rebased (and see other new PR's) >> fail with: >> >> * What went wrong:*15:30:40* Execution failed for task >> ':sdks:java:io:thrift:compileThrift'.*15:30:40* > A problem occurred >> starting process 'command 'thrift'' >> >> >> _/ >> _/ Alex Van Boxel >> >
