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

Posted to github@arrow.apache.org by "Turbo87 (via GitHub)" <gi...@apache.org> on 2023/07/10 08:49:49 UTC

[GitHub] [arrow-rs] Turbo87 commented on issue #4496: object_store/InMemory: Make `clone()` non-async

Turbo87 commented on issue #4496:
URL: https://github.com/apache/arrow-rs/issues/4496#issuecomment-1628521794

   On second thought I'm even more confused by the current `clone()` implementation now. It seems that it copies the data instead of creating a second instance of `InMemory` sharing the backing store with the first instance. Maybe it would be better to rename `clone()` to e.g. `fork()` and use a derived `Clone` implementation, which just cheaply clones the `Arc` of the backing store. 🤔 


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