Skip to content
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

Remove arrow dependency #16640

Merged
merged 52 commits into from
Aug 27, 2024
Merged
Changes from 1 commit
Commits
Show all changes
52 commits
Select commit Hold shift + click to select a range
a43e7f2
Remove libarrow dependency from libcudf and migrate to tests only
vyasr Aug 20, 2024
d1d23dd
Remove libarrow dependency from all Cython builds
vyasr Aug 20, 2024
424993f
Lots of cleanup and simplification of get_arrow.cmake
vyasr Aug 20, 2024
a1bf94f
Move module to tests
vyasr Aug 20, 2024
36a86b5
Remove now unnecessary args
vyasr Aug 20, 2024
aff2a66
Remove one unnecessary branch
vyasr Aug 20, 2024
37d2793
Remove some unnecessary specializations for some targets
vyasr Aug 20, 2024
07547b0
Switch to static arrow
vyasr Aug 20, 2024
14391ea
Clean out some unnecessary bits
vyasr Aug 20, 2024
2d544a1
Stop exporting since now it's a private dependency
vyasr Aug 20, 2024
d865616
Some variable simplification
vyasr Aug 20, 2024
3fe333d
Set Arrow_FOUND at the end
vyasr Aug 20, 2024
5040b5d
Remove now unused code string
vyasr Aug 20, 2024
e84e26b
Combine some conditions
vyasr Aug 20, 2024
9222d15
Reenable linting rules
vyasr Aug 20, 2024
c129ec7
Remove libarrow from cpp build reqs
vyasr Aug 20, 2024
282da76
Remove pyarrow from Python build deps
vyasr Aug 20, 2024
614992c
Drop numpy as a build dependency
vyasr Aug 20, 2024
987fdf0
Add get_arrow.cmake to JNI build
vyasr Aug 21, 2024
c8dce44
Stop installing arrow into wheels
vyasr Aug 22, 2024
6eb7e99
Revert changes to get_arrow.cmake
vyasr Aug 22, 2024
3ec9ee5
Disable libarrow from pyarrow
vyasr Aug 22, 2024
2e1c11e
Remove I/O and Python options
vyasr Aug 22, 2024
c6c4ba6
Remove no longer support armv8 option
vyasr Aug 22, 2024
c8b8e42
Remove export logic since we no longer install
vyasr Aug 22, 2024
4664212
Port changes over to Java copy
vyasr Aug 22, 2024
b5721cf
Two more tests need Arrow
vyasr Aug 22, 2024
ec9105b
Add Arrow dep to example and default Arrow linkage to static everywhere
vyasr Aug 22, 2024
e90fdf8
Add build type specifier
vyasr Aug 22, 2024
8af5b33
Temporarily disable interop example to validate everything else
vyasr Aug 22, 2024
b9cf424
Set the build type correctly
vyasr Aug 22, 2024
a6e8528
Go back to a single get_arrow.cmake
vyasr Aug 22, 2024
923d1f2
Update comment for interop example
vyasr Aug 22, 2024
351c358
Stop passing nonexistent args
vyasr Aug 22, 2024
bb5b267
Default to excluding Arrow from installation
vyasr Aug 22, 2024
9c3dc42
Merge remote-tracking branch 'upstream/branch-24.10' into feat/remove…
vyasr Aug 23, 2024
3730e99
Merge remote-tracking branch 'upstream/branch-24.10' into feat/remove…
vyasr Aug 26, 2024
d1d5518
Fix numpy pinning
vyasr Aug 26, 2024
433ba08
Add back arrow parquet support
vyasr Aug 26, 2024
2881ea4
Merge remote-tracking branch 'upstream/branch-24.10' into feat/remove…
vyasr Aug 26, 2024
602c4d6
Fix order
vyasr Aug 26, 2024
04d6e4f
Add missing set
vyasr Aug 26, 2024
556c2b5
Remove extra parquet options altogether
vyasr Aug 26, 2024
9bc6450
Revert "Remove extra parquet options altogether"
vyasr Aug 26, 2024
afa8cb7
Remove one more set of unnecessary libarrow deps
vyasr Aug 26, 2024
213f0db
Make sure install rules are in place for the JNI
vyasr Aug 26, 2024
d3e14e0
Revert unconditional parquet inclusion
vyasr Aug 26, 2024
15ecf66
Make sure boost is avialable for the java test build
vyasr Aug 26, 2024
4e653e9
Put back install rules
vyasr Aug 26, 2024
894ac93
Generate the install rules conditionally
vyasr Aug 27, 2024
aa2952f
Include rapids-export
vyasr Aug 27, 2024
29833e6
Merge remote-tracking branch 'upstream/branch-24.10' into feat/remove…
vyasr Aug 27, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove some unnecessary specializations for some targets
  • Loading branch information
vyasr committed Aug 22, 2024
commit 37d2793693607238ec501421d21264313e1010a5
18 changes: 0 additions & 18 deletions cpp/tests/cmake/thirdparty/get_arrow.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -155,26 +155,8 @@ function(find_and_configure_arrow VERSION BUILD_STATIC)
if (TARGET cudf::arrow_static AND (NOT TARGET arrow_static))
add_library(arrow_static ALIAS cudf::arrow_static)
endif()
if (NOT TARGET arrow::flatbuffers)
add_library(arrow::flatbuffers INTERFACE IMPORTED)
endif()
if (NOT TARGET arrow::hadoop)
add_library(arrow::hadoop INTERFACE IMPORTED)
endif()
]=]
)
if(NOT TARGET xsimd)
string(
APPEND
arrow_code_string
"
if(NOT TARGET arrow::xsimd)
add_library(arrow::xsimd INTERFACE IMPORTED)
target_include_directories(arrow::xsimd INTERFACE \"${Arrow_BINARY_DIR}/xsimd_ep/src/xsimd_ep-install/include\")
endif()
"
)
endif()
rapids_cmake_install_lib_dir(lib_dir)
if(TARGET arrow_static)
get_target_property(interface_libs arrow_static INTERFACE_LINK_LIBRARIES)
Expand Down