https://bugs.openldap.org/show_bug.cgi?id=10395
--- Comment #26 from kero renault.cle@gmail.com --- Hello Howard,
Thank you very much for the much cleaner version of this patch. I created two dummy PRs to test the engine with it, as it extensively uses the nested read transaction feature.
One of the PRs [mdb.master3 PR] is to check the patch on mdb.master3, and it seems at least one thing is working differently from mdb.master. I haven't checked what it was, but I doubt it is related to this patch, since it's affecting an env that doesn't use the new feature.
[mdb.master3 PR]: https://github.com/meilisearch/meilisearch/pull/6309
The other PR [mdb.master PR] is testing your rebased patch against mdb.master. The reason I still need it on mdb.master is that we can't make a breaking change to the database format, and I prefer that Meilisearch rely on the mainstream LMDB with a single derivation rather than a completely different version (mdb.master3). I will maintain a fork and set up a notification system to be informed of new LMDB releases (emails are great; combining them with issues is better).
[mdb.master PR]: https://github.com/meilisearch/meilisearch/pull/6310
Anyway, the patch seems valid and doesn't trigger any issues on the mdb.master branch. On the mdb.master3 branch I would consider it valid too, the problem is unrelated to this very patch.
Have a good weekend, and thanks for your time, kero