jorisvandenbossche opened a new pull request, #39783:
URL: https://github.com/apache/arrow/pull/39783

   ### Rationale for this change
   
   The Array exporter currently uses `buffer->data()`, which is documented as 
"The buffer has to be a CPU buffer (`is_cpu()` is true). Otherwise, an 
assertion may be thrown or a null pointer may be returned.". 
   In practice, we indeed return null for non-CPU data. There is only a debug 
check that asserts the device. 
   
   This means that for non-CPU data the C device interface is currently 
returning null pointers for the buffers.
   
   ### What changes are included in this PR?
   
   Use the buffer's `address()` instead of `data()`
   
   ### Are these changes tested?
   
   Not yet
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to