Description
ConcurrentAccessTest uses an external std::mutex to serialize access. This proves the server's external lock works, but doesn't test the engine's own rw_mutex. If the server-level lock is ever removed, data races go undetected.
Recommended Fix
Add a test that exercises the engine's internal locking by calling insert() and search() from different threads without the external mutex (using std::shared_mutex directly).
Description
ConcurrentAccessTestuses an externalstd::mutexto serialize access. This proves the server's external lock works, but doesn't test the engine's ownrw_mutex. If the server-level lock is ever removed, data races go undetected.Recommended Fix
Add a test that exercises the engine's internal locking by calling
insert()andsearch()from different threads without the external mutex (usingstd::shared_mutexdirectly).