https://bugs.openldap.org/show_bug.cgi?id=10395
kero renault.cle@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #1096|0 |1 is obsolete| |
--- Comment #8 from kero renault.cle@gmail.com --- Created attachment 1098 --> https://bugs.openldap.org/attachment.cgi?id=1098&action=edit Allow multiple nested read transactions from a write transaction using a mutex
Hello Howard, and thank you for the early review,
I found the time to replace the C11 atomics with C99 mutexes. However, I am not entirely sure if this is the correct way to use mutexes in this code, or if we should instead use macros or defines to ensure compatibility across all platforms. Anyway, this code is relatively small, and it would be easy to patch. It compiles and works on all platforms, as confirmed by the numerous CI tests we have on Heed [1] and Meilisearch [2].
Thank you very much, Have a nice day
[1]: https://github.com/meilisearch/heed/pull/307/commits/464a9c33f76e0679115eef7... [2]: https://github.com/meilisearch/meilisearch/pull/6042/checks