Skip to content

ARROW-6330: [C++] Include missing API headers#5175

Closed
emkornfield wants to merge 1 commit intoapache:masterfrom
emkornfield:add_missing_public_api_header
Closed

ARROW-6330: [C++] Include missing API headers#5175
emkornfield wants to merge 1 commit intoapache:masterfrom
emkornfield:add_missing_public_api_header

Conversation

@emkornfield
Copy link
Copy Markdown
Contributor

No description provided.

@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5175      +/-   ##
==========================================
+ Coverage   87.36%   89.25%   +1.88%     
==========================================
  Files         909      739     -170     
  Lines      122252   105406   -16846     
  Branches     1437        0    -1437     
==========================================
- Hits       106807    94076   -12731     
+ Misses      15083    11330    -3753     
+ Partials      362        0     -362
Impacted Files Coverage Δ
go/arrow/math/uint64_amd64.go
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
rust/parquet/src/arrow/converter.rs
go/arrow/array/builder.go
js/src/util/fn.ts
... and 461 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 6cbaf72...c95ebcc. Read the comment docs.

@wesm wesm changed the title ARROW-6330: Include missing API headers ARROW-6330: [C++] Include missing API headers Aug 23, 2019
Copy link
Copy Markdown
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

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.

3 participants