-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[C++] -DARROW_HDFS may add needless system Protobuf dependency for static linking #39136
Comments
kou
added a commit
to kou/arrow
that referenced
this issue
Dec 8, 2023
kou
added a commit
that referenced
this issue
Dec 8, 2023
…OW_HDFS=ON (#39137) ### Rationale for this change Our HDFS related codes don't depend on Protobuf because we process HDFS via external `libhdfs.so` and it's `dlopen()`-ed. ### What changes are included in this PR? Remove a needless CMake configuration. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * Closes: #39136 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
mapleFU
pushed a commit
to mapleFU/arrow
that referenced
this issue
Dec 13, 2023
… -DARROW_HDFS=ON (apache#39137) ### Rationale for this change Our HDFS related codes don't depend on Protobuf because we process HDFS via external `libhdfs.so` and it's `dlopen()`-ed. ### What changes are included in this PR? Remove a needless CMake configuration. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * Closes: apache#39136 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
clayburn
pushed a commit
to clayburn/arrow
that referenced
this issue
Jan 23, 2024
… -DARROW_HDFS=ON (apache#39137) ### Rationale for this change Our HDFS related codes don't depend on Protobuf because we process HDFS via external `libhdfs.so` and it's `dlopen()`-ed. ### What changes are included in this PR? Remove a needless CMake configuration. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * Closes: apache#39136 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
dgreiss
pushed a commit
to dgreiss/arrow
that referenced
this issue
Feb 19, 2024
… -DARROW_HDFS=ON (apache#39137) ### Rationale for this change Our HDFS related codes don't depend on Protobuf because we process HDFS via external `libhdfs.so` and it's `dlopen()`-ed. ### What changes are included in this PR? Remove a needless CMake configuration. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * Closes: apache#39136 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug, including details regarding any error messages, version, and platform.
See https://github.com/apache/arrow/pull/13911/files#r1418811551 for the related change.
We may be able to just remove the change because we already have a related code:
arrow/cpp/CMakeLists.txt
Lines 775 to 778 in 081b402
Component(s)
C++
The text was updated successfully, but these errors were encountered: