Skip to content

Conversation

@xhochy
Copy link
Member

@xhochy xhochy commented Jun 29, 2019

C++ and Python Tests pass locally so this seems to be ok for us.

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4752      +/-   ##
==========================================
+ Coverage   86.44%   88.93%   +2.49%     
==========================================
  Files         992      717     -275     
  Lines      138020    98865   -39155     
  Branches     1418        0    -1418     
==========================================
- Hits       119311    87926   -31385     
+ Misses      18347    10939    -7408     
+ Partials      362        0     -362
Impacted Files Coverage Δ
cpp/src/plasma/thirdparty/ae/ae.c 70.75% <0%> (-0.95%) ⬇️
cpp/src/arrow/util/thread-pool-test.cc 97.66% <0%> (-0.94%) ⬇️
go/arrow/ipc/writer.go
js/src/util/fn.ts
go/arrow/memory/memory_avx2_amd64.go
rust/datafusion/src/execution/filter.rs
rust/arrow/src/csv/writer.rs
rust/datafusion/src/bin/main.rs
go/arrow/ipc/file_reader.go
js/src/builder/index.ts
... and 267 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 4d25902...6e087d6. Read the comment docs.

Copy link
Member

@wesm wesm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@wesm wesm closed this in 10083ab Jun 30, 2019
@wesm wesm deleted the ARROW-5609 branch June 30, 2019 19:05
kou pushed a commit that referenced this pull request Jul 4, 2019
C++ and Python Tests pass locally so this seems to be ok for us.

Author: Uwe L. Korn <uwelk@xhochy.com>

Closes #4752 from xhochy/ARROW-5609 and squashes the following commits:

6e087d6 <Uwe L. Korn> ARROW-5609:  Set CMP0068 CMake policy to avoid macOS warnings
wesm pushed a commit that referenced this pull request Jul 13, 2019
C++ and Python Tests pass locally so this seems to be ok for us.

Author: Uwe L. Korn <uwelk@xhochy.com>

Closes #4752 from xhochy/ARROW-5609 and squashes the following commits:

6e087d6 <Uwe L. Korn> ARROW-5609:  Set CMP0068 CMake policy to avoid macOS warnings
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.93%. Comparing base (4d25902) to head (6e087d6).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4752      +/-   ##
==========================================
+ Coverage   86.44%   88.93%   +2.49%     
==========================================
  Files         992      717     -275     
  Lines      138020    98865   -39155     
  Branches     1418        0    -1418     
==========================================
- Hits       119311    87926   -31385     
+ Misses      18347    10939    -7408     
+ Partials      362        0     -362     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

4 participants