search

automerge / automerge-repo-rs

MIT License
39 stars 6 forks source link

Consider `Arc<RwLock<Doc>>` instead of `Arc<Mutex<Doc>>` #24

Closed teohhanhui closed 1 year ago

teohhanhui commented 1 year ago

In the implementation of our ORM layer on top of autoserde, we're using Arc<RwLock<Doc>> to support multiple concurrent readers.

I wonder if it'd be appropriate for this crate?

alexjg commented 1 year ago

This is implemented in #20 which I think we will merge shortly

gterzian commented 1 year ago

Done in https://github.com/automerge/spanreed/pull/20