Skip to content

ARROW-7036: [C++] Version up ORC to avoid compile errors#5762

Closed
mrkn wants to merge 1 commit intoapache:masterfrom
mrkn:ARROW-7036
Closed

ARROW-7036: [C++] Version up ORC to avoid compile errors#5762
mrkn wants to merge 1 commit intoapache:masterfrom
mrkn:ARROW-7036

Conversation

@mrkn
Copy link
Copy Markdown
Member

@mrkn mrkn commented Oct 31, 2019

ORC 1.5.5 has some compile errors occurred on Xcode 11.
The version 1.5.7 resolves them.

@mrkn mrkn requested a review from kou October 31, 2019 04:47
@github-actions
Copy link
Copy Markdown

@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5762      +/-   ##
==========================================
+ Coverage   88.99%   89.54%   +0.54%     
==========================================
  Files        1006      814     -192     
  Lines      137241   121731   -15510     
  Branches     1501        0    -1501     
==========================================
- Hits       122134   108998   -13136     
+ Misses      14742    12733    -2009     
+ Partials      365        0     -365
Impacted Files Coverage Δ
cpp/src/arrow/util/thread_pool_test.cc 97.66% <0%> (-0.94%) ⬇️
python/pyarrow/tests/test_parquet.py 95.24% <0%> (-0.06%) ⬇️
go/arrow/ipc/writer.go
go/arrow/math/uint64_amd64.go
go/arrow/memory/memory_avx2_amd64.go
go/arrow/ipc/file_reader.go
js/src/builder/index.ts
js/src/enum.ts
go/arrow/array/builder.go
js/src/util/fn.ts
... and 184 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 86f34aa...95a70e7. Read the comment docs.

Copy link
Copy Markdown
Member

@kou kou left a comment

Choose a reason for hiding this comment

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

+1

@kou kou closed this in c303f6e Oct 31, 2019
@mrkn mrkn deleted the ARROW-7036 branch October 31, 2019 07:08
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