New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARROW-7170: [C++] Fix linking with bundled ORC #5834
Conversation
c5ae648
to
57e811b
Compare
GTest::GTest | ||
Snappy::snappy | ||
LZ4::lz4 | ||
ZLIB::ZLIB) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need ARROW_WITH_XXX
check?
For example:
if(ARROW_WITH_LZ4)
list(APPEND ORC_MIN_TEST_LIBS LZ4::lz4)
endif()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, I don't think so? LZ4 is not optional when building ORC, AFAICT.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And we have this in cpp/CMakeLists.txt
:
if(ARROW_ORC)
set(ARROW_WITH_LZ4 ON)
set(ARROW_WITH_SNAPPY ON)
set(ARROW_WITH_ZLIB ON)
endif()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. I understand.
Note I still get an error later: ``` [libprotobuf ERROR google/protobuf/descriptor_database.cc:120] File already exists in database: orc_proto.proto [libprotobuf FATAL google/protobuf/descriptor.cc:1359] CHECK failed: GeneratedDatabase()->Add(encoded_file_descriptor, size): terminate called after throwing an instance of 'google::protobuf::FatalException' what(): CHECK failed: GeneratedDatabase()->Add(encoded_file_descriptor, size): Abandon ```
57e811b
to
65c1c5d
Compare
Rebased. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
GTest::GTest | ||
Snappy::snappy | ||
LZ4::lz4 | ||
ZLIB::ZLIB) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. I understand.
Note I still get an error later: