https://bugs.openldap.org/show_bug.cgi?id=10395

kero <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #1096|0                           |1
        is obsolete|                            |

--- Comment #8 from kero <[email protected]> ---
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/464a9c33f76e0679115eef7379c9bdde9d5e1eda
[2]: https://github.com/meilisearch/meilisearch/pull/6042/checks

-- 
You are receiving this mail because:
You are on the CC list for the issue.

Reply via email to