Jefffrey commented on issue #10253:
URL: https://github.com/apache/arrow-rs/issues/10253#issuecomment-5067325632

   im not as acquainted with the ffi side of things, but whats stated here 
makes sense. given it seems like a minor issue i suppose we can just wait for 
the next major release.
   
   that being said, i believe a bigger issue is that using `release` in the 
drop implementation has the same issue but with no easy way to slap an `unsafe` 
on it to make it as such 🤔 
   
   
https://github.com/apache/arrow-rs/blob/f7dfcd25aabeb01641fe4b6c35ab964fdf0b24aa/arrow-array/src/ffi_stream.rs#L99-L111
   
   
https://github.com/apache/arrow-rs/blob/f7dfcd25aabeb01641fe4b6c35ab964fdf0b24aa/arrow-array/src/ffi_stream.rs#L164-L171


-- 
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