apache / parquet-cpp / 707

Ran 01 Jul 2016 09:52PM UTC

Jobs 2

Files 106

Run time 1min

Badge

Embed ▾

pending completion

Build # 707

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

4762 of 4983 relevant lines covered (95.56%)

20784.96 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	707.1	01 Jul 2016 09:54PM UTC	0	95.56	Travis Job 707.1
2	707.2	01 Jul 2016 09:52PM UTC	0	0.0	Travis Job 707.2

Source Files on build 707

Detailed source file information is not available for this build.

apache / parquet-cpp / 707

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 707

apache / parquet-cpp / 707

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 707

README BADGES
x