[PARQUET-1654] Remove unnecessary options when building thrift

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.11.0
• Component/s: parquet-thrift
• Labels: pull-request-available
• Assignee: Kengo Seki
• Reporter: Kengo Seki
• Created: 18/Sep/19 04:38
• Updated: 21/Sep/19 07:30
• Resolved: 21/Sep/19 07:30

Description

In dev/travis-before_install.sh, configure options for thrift are defined as follows:

./configure --disable-gen-erl --disable-gen-hs --without-ruby --without-haskell --without-erlang --without-php --without-nodejs

And the following libraries are enabled on Travis:

thrift 0.12.0

Building C (GLib) Library .... : yes
Building C# (Mono) Library ... : no
Building C++ Library ......... : yes
Building Common Lisp Library.. : no
Building D Library ........... : no
Building Dart Library ........ : no
Building dotnetcore Library .. : no
Building Erlang Library ...... : no
Building Go Library .......... : yes
Building Haskell Library ..... : no
Building Haxe Library ........ : no
Building Java Library ........ : yes
Building Lua Library ......... : no
Building NodeJS Library ...... : no
Building Perl Library ........ : no
Building PHP Library ......... : no
Building Plugin Support ...... : no
Building Python Library ...... : yes
Building Py3 Library ......... : yes
Building Ruby Library ........ : no
Building Rust Library ........ : no

But Parquet requires only the C++ library to be installed, so we can disable C, Go, Java, and Python2 (Python3 doesn't seem to be disabled). This will save about 30 seconds on each CI run.

Issue Links

• links to: GitHub Pull Request #676