apache / arrow-nanoarrow

Helpers for Arrow C Data & Arrow C Stream interfaces
https://arrow.apache.org/nanoarrow
Apache License 2.0
169 stars 35 forks source link

refactor(extensions/nanoarrow_device): Migrate CUDA device implementation to use the driver API #488

Closed paleolimbot closed 4 months ago

paleolimbot commented 4 months ago

Closes #246.

This PR doesn't change much about the existing implementation (and I think there are some things that need to be changed!), it just eliminates the dependency on the runtime library. The driver API is a better fit here anyway since we're doing very low-level things!

This isn't tested in CI yet (working on that here: https://github.com/apache/arrow-nanoarrow/pull/490 ).

paleolimbot commented 4 months ago

I'm going to merge this to get started on follow-up work, but feel free to give any comments on this diff and I'll incorporate them into the next PR!