You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@arrow.apache.org by "Antoine Pitrou (JIRA)" <ji...@apache.org> on 2018/10/09 15:03:00 UTC

[jira] [Commented] (ARROW-3442) [C++] Use dynamic linking for unit tests, ensure coverage working properly with clang

    [ https://issues.apache.org/jira/browse/ARROW-3442?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16643581#comment-16643581 ] 

Antoine Pitrou commented on ARROW-3442:
---------------------------------------

I started looking into this.
* with a modern gcc (gcc 7.3.0 on Ubuntu 18.04), coverage info is generated properly through .so files (but perhaps the gcc version on Travis is too old?)
* collecting coverage generated with clang needs a small hack: http://logan.tw/posts/2015/04/28/check-code-coverage-with-clang-and-lcov/


> [C++] Use dynamic linking for unit tests, ensure coverage working properly with clang
> -------------------------------------------------------------------------------------
>
>                 Key: ARROW-3442
>                 URL: https://issues.apache.org/jira/browse/ARROW-3442
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Wes McKinney
>            Priority: Major
>             Fix For: 0.12.0
>
>
> See experience around this in Apache Kudu
> https://github.com/apache/kudu/commit/48799d3fed53c79a5163fd9c4cd9cbae80a9f2d6



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)