You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/06/24 20:15:59 UTC

[GitHub] [arrow-rs] jhorstmann opened a new issue, #1944: Send + Sync impl for Allocation might be unsound

jhorstmann opened a new issue, #1944:
URL: https://github.com/apache/arrow-rs/issues/1944

   **Describe the bug**
   
   As noted by @jorgecarleitao in https://github.com/jorgecarleitao/arrow2/pull/1099#issuecomment-1165186016 the unsafe impl of `Send + Sync` for [`Buffer`](https://github.com/apache/arrow-rs/blob/c5cf2f21a59bec896b1db48f97b8aa0faacf624b/arrow/src/buffer/immutable.rs#L260) might not be sound if the custom [`Allocation`](https://github.com/apache/arrow-rs/blob/c5cf2f21a59bec896b1db48f97b8aa0faacf624b/arrow/src/alloc/mod.rs#L130) is not also `Send + Sync`.
   
   
   **Expected behavior**
   
   `Allocation` should require `RefUnwindSafe + Send + Sync`. `Bytes` probably still needs an unsafe `Send + Sync` impl, but `Buffer` should then be automatically `Send + Sync`.
   


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-rs] tustvold closed issue #1944: Send + Sync impl for Allocation might be unsound

Posted by GitBox <gi...@apache.org>.

tustvold closed issue #1944: Send + Sync impl for Allocation might be unsound
URL: https://github.com/apache/arrow-rs/issues/1944


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org