apache / parquet-cpp / 707 / 2

Ran 01 Jul 2016 09:52PM UTC

Files 106

Run time 4s

Badge

Embed ▾

Committed 01 Jul 2016 09:45PM UTC coverage: 0.0%. Remained the same

Job # 707.2

Build Type

push

travis-ci

Committed by

wesm

Commit Message

PARQUET-489: Shared library symbol visibility

This was a bit of rabbit hole, because on Linux you must instruct gcc how to hide symbols from your static thirdparty libraries (see `src/parquet/symbols.map`). I learned about this from Apache Kudu (incubating) client cmake files.

Probably still a bit more to do here (and I may have missed some things we should export, and exporting things that should not be exported); I will make sure the Arrow-Parquet tests still pass, but wanted to get some early feedback.

Author: Wes McKinney <wesm@apache.org>

Closes #131 from wesm/PARQUET-489 and squashes the following commits:

2cd834a [Wes McKinney] Link static libs to all executables
ed47d1e [Wes McKinney] Hide more symbols when statically-linking libstdc++
d052387 [Wes McKinney] Correctly export default_allocator
9f17ea3 [Wes McKinney] Install visibility.h
9d97985 [Wes McKinney] - Adapt symbol.map from Kudu to hide thirdparty static deps - Link tests to static lib - Add visibility header - First cut exports

Run Details

0 of 0 relevant lines covered (NaN%)

0.0 hits per line

apache / parquet-cpp / 707 / 2

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 707.2

apache / parquet-cpp / 707 / 2

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 707.2

README BADGES
x