[jira] [Commented] (ARROW-27) C++: Upload gcov coverage data to coveralls.io

2018-01-23 Thread Wes McKinney (JIRA)

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

Wes McKinney commented on ARROW-27:
---

It would be nice to be able to look at C++ code coverage. If someone can 
investigate this further it would be a big help

> C++: Upload gcov coverage data to coveralls.io
> --
>
> Key: ARROW-27
> URL: https://issues.apache.org/jira/browse/ARROW-27
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: C++
>Reporter: Wes McKinney
>Assignee: Wes McKinney
>Priority: Major
>  Labels: pull-request-available
>
> I recently did this for Parquet 
> (https://github.com/apache/parquet-cpp/commit/9cab887f2337ae6205cbd7ea908bcb749c5c9343);
>  it will be nice to have the coverage report available to assist with unit 
> testing.



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


[jira] [Commented] (ARROW-27) C++: Upload gcov coverage data to coveralls.io

2017-11-09 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on ARROW-27:
-

wesm commented on issue #1241: ARROW-27: [C++] Set up C++ code coverage uploads 
to coveralls.io
URL: https://github.com/apache/arrow/pull/1241#issuecomment-343273846
 
 
   Any more ideas on this? Would be nice to get this working soon to start 
looking at coverage reports


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> C++: Upload gcov coverage data to coveralls.io
> --
>
> Key: ARROW-27
> URL: https://issues.apache.org/jira/browse/ARROW-27
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: C++
>Reporter: Wes McKinney
>Assignee: Wes McKinney
>  Labels: pull-request-available
> Fix For: 0.8.0
>
>
> I recently did this for Parquet 
> (https://github.com/apache/parquet-cpp/commit/9cab887f2337ae6205cbd7ea908bcb749c5c9343);
>  it will be nice to have the coverage report available to assist with unit 
> testing.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (ARROW-27) C++: Upload gcov coverage data to coveralls.io

2017-10-28 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on ARROW-27:
-

xhochy commented on issue #1241: ARROW-27: [C++] Set up C++ code coverage 
uploads to coveralls.io
URL: https://github.com/apache/arrow/pull/1241#issuecomment-340185734
 
 
   Seems like there is still something missing: 
https://coveralls.io/builds/13886247/source?filename=src%2Farrow%2Fvisitor.cc


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> C++: Upload gcov coverage data to coveralls.io
> --
>
> Key: ARROW-27
> URL: https://issues.apache.org/jira/browse/ARROW-27
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: C++
>Reporter: Wes McKinney
>Assignee: Wes McKinney
>  Labels: pull-request-available
> Fix For: 0.8.0
>
>
> I recently did this for Parquet 
> (https://github.com/apache/parquet-cpp/commit/9cab887f2337ae6205cbd7ea908bcb749c5c9343);
>  it will be nice to have the coverage report available to assist with unit 
> testing.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (ARROW-27) C++: Upload gcov coverage data to coveralls.io

2017-10-23 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on ARROW-27:
-

wesm opened a new pull request #1241: ARROW-27: [C++] Set up C++ code coverage 
uploads to coveralls.io
URL: https://github.com/apache/arrow/pull/1241
 
 
   May take a few rounds of trial and error to get this working properly


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> C++: Upload gcov coverage data to coveralls.io
> --
>
> Key: ARROW-27
> URL: https://issues.apache.org/jira/browse/ARROW-27
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: C++
>Reporter: Wes McKinney
>Assignee: Wes McKinney
>  Labels: pull-request-available
> Fix For: 0.8.0
>
>
> I recently did this for Parquet 
> (https://github.com/apache/parquet-cpp/commit/9cab887f2337ae6205cbd7ea908bcb749c5c9343);
>  it will be nice to have the coverage report available to assist with unit 
> testing.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (ARROW-27) C++: Upload gcov coverage data to coveralls.io

2017-10-18 Thread Wes McKinney (JIRA)

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

Wes McKinney commented on ARROW-27:
---

Having C++ coverage reports is long overdue. [~pcmoritz] [~xhochy] [~cpcloud] 
anyone up to do this? The biggest thing will be dealing with the exclusions 
when uploading to coveralls. We already have this set up for parquet-cpp but I 
haven't checked the report in a while to see if it looks reasonable (sometimes 
a portion of the STL is picked up)

> C++: Upload gcov coverage data to coveralls.io
> --
>
> Key: ARROW-27
> URL: https://issues.apache.org/jira/browse/ARROW-27
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: C++
>Reporter: Wes McKinney
> Fix For: 0.8.0
>
>
> I recently did this for Parquet 
> (https://github.com/apache/parquet-cpp/commit/9cab887f2337ae6205cbd7ea908bcb749c5c9343);
>  it will be nice to have the coverage report available to assist with unit 
> testing.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)