Skip to content

ARROW-3000: [C++] Add option to label test groups then only build those unit tests#2406

Closed
wesm wants to merge 3 commits intoapache:masterfrom
wesm:ARROW-3000
Closed

ARROW-3000: [C++] Add option to label test groups then only build those unit tests#2406
wesm wants to merge 3 commits intoapache:masterfrom
wesm:ARROW-3000

Conversation

@wesm
Copy link
Copy Markdown
Member

@wesm wesm commented Aug 8, 2018

This should help make our groups of unit tests more modular and easier to build only the relevant ones for a particular test cycle

wesm added 2 commits August 8, 2018 15:42
…t in the cmake configuration. This way we can choose to only build python-test in Travis CI, for example

Change-Id: I3b5618f7fe301fc88704cd5895689759a58b3659
Change-Id: I55094e7efb6a9e7cff3093e17f13b1fbdbc9241d
@wesm
Copy link
Copy Markdown
Member Author

wesm commented Aug 8, 2018

This isn't quite working yet, I'll take a look:

0.16s$ $TRAVIS_BUILD_DIR/ci/travis_script_cpp.sh
~/build/apache/arrow/cpp-build ~/build/apache/arrow
Test project /home/travis/build/apache/arrow/cpp-build
No tests were found!!!
~/build/apache/arrow
~/build/apache/arrow ~/build/apache/arrow
geninfo: ERROR: no .gcda files found in .!
~/build/apache/arrow

Change-Id: Iba9dcd5d40c8a445842b1463d0667ab1fd6a37e4
@wesm
Copy link
Copy Markdown
Member Author

wesm commented Aug 8, 2018

The C++/Python build ran in 31 minutes this time

@codecov-io
Copy link
Copy Markdown

codecov-io commented Aug 8, 2018

Codecov Report

Merging #2406 into master will increase coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2406      +/-   ##
==========================================
+ Coverage   84.67%   84.73%   +0.06%     
==========================================
  Files         293      293              
  Lines       45459    45474      +15     
==========================================
+ Hits        38492    38533      +41     
+ Misses       6926     6904      -22     
+ Partials       41       37       -4
Impacted Files Coverage Δ
go/arrow/math/uint64_sse4_amd64.go 0% <0%> (-100%) ⬇️
go/arrow/math/int64_sse4_amd64.go 0% <0%> (-100%) ⬇️
go/arrow/math/float64_sse4_amd64.go 0% <0%> (-100%) ⬇️
go/arrow/memory/memory_sse4_amd64.go 0% <0%> (-100%) ⬇️
python/pyarrow/plasma.py 87.5% <0%> (-0.22%) ⬇️
go/arrow/math/float64_amd64.go 33.33% <0%> (ø) ⬆️
go/arrow/math/int64_amd64.go 33.33% <0%> (ø) ⬆️
go/arrow/math/uint64_amd64.go 33.33% <0%> (ø) ⬆️
cpp/src/plasma/store.cc 89.64% <0%> (+0.42%) ⬆️
go/arrow/math/math_amd64.go 36.84% <0%> (+5.26%) ⬆️
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c97cd6...1db587d. Read the comment docs.

@wesm
Copy link
Copy Markdown
Member Author

wesm commented Aug 9, 2018

+1

@wesm wesm closed this in 699b0d2 Aug 9, 2018
@wesm wesm deleted the ARROW-3000 branch August 9, 2018 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants