kanidm / concread

Concurrently Readable Data Structures for Rust
Mozilla Public License 2.0
343 stars 15 forks source link

Extend Serde support to cover asynch maps. #109

Closed adamreichold closed 8 months ago

adamreichold commented 8 months ago

The Serialize implementation targets the read transactions instead of the collection, as we cannot transparently await from within the trait.

This is somewhat embarrassing as I even asked for point release which you graciously provided only to realise at the office that we are using the "asynch" versions so that my previous do not actually cover our use case. 🤦🏽 I am terribly sorry for the extra work.

Firstyear commented 8 months ago

Don't worry about it mate, appreciate the PR's. The release is pretty quick anyway, so I'll merge this and will release again. Thanks heaps :)