David Wendt
David Wendt
There is already a `to_arrow_utilities.cpp/.hpp`. Could we just move the function declared/defined there into this new `arrow_utilities.cpp/.hpp`? And then delete the `to_arrow_utilities.cpp/.hpp` files.
Looks like you will need to update these https://github.com/rapidsai/cudf/blob/8b5ff188e79bb79ca0c2d581e94d3a91654a2d31/cpp/tests/interop/from_arrow_device_test.cpp#L52 to expect the `std::invalid_argument` exception instead of `cudf::logic_error`
I don't think this requires new APIs to libcudf. The `cudf::column` and `cudf::table` are [data-owning structs](https://github.com/rapidsai/cudf/blob/branch-24.04/cpp/doxygen/developer_guide/DEVELOPER_GUIDE.md#cudfcolumn) in libcudf. For zero-copy you should be able wrap the arrow data in device...
Ok. That makes sense. This appears to be just a wrapper around the `cudf::column_view` [constructor](https://github.com/rapidsai/cudf/blob/57bbe94e995b9a0365276e4cb26853dce219e22a/cpp/include/cudf/column/column_view.hpp#L371-L377). But resolving the data-type and the other components needed for the parameters would involve significant...