Solve issues with mutexes while running tests on MacOS M1

[JoanFM]

Describe the changes in the pull request

2 issues are handled in this PR:

1- An abort was identified while running a test related to PyBatchIterator. The abort happened when the test was being shutdown. The error comes from the deleterpassed to the PyBatchIterator shared_ptr which captures the PyIndexby reference. The problem is that there is no guarantee that the PyIndexlifetime will survive the Iterator. In that deleter a mutex is released (which could be in an invalid state). The solution is to have the mutexbe wrapped in a shared_ptr that will share the ownership with the PyBatchIterator deleter as it captures it by value (incrementing its reference count).

2- Deadlock found on tests/flow/test_hnsw_parallel.py::test_parallel_insert_batch_search:

• Thread #N-1 has the indexGuard locked in shared mode, and tries to recover the GIL in the getNextResult
• Thread #N tries to acquire indexGuard exclusively, while N-1 has the indexGuard in shared mode. (WAIT)
• Thread #N+1 is another iterator that want indexGuard in shared mode (It goes in the queue) but cannot get it in shared mode (I guess is System dependant if they would schedule a reader to bypass the writer ) potential starvation. Thread N+1 has the GIL.

This is solved by releasing the GIL in Thread N+1 before asking for indexGuard.

[CLAassistant]

All committers have signed the CLA.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.24%. Comparing base (e4fc3e7) to head (7ac126c).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #677   +/-   ##
=======================================
  Coverage   96.24%   96.24%           
=======================================
  Files         112      112           
  Lines        6278     6278           
=======================================
  Hits         6042     6042           
  Misses        236      236           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

Backport failed for 0.7, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin 0.7
git worktree add -d .worktree/backport-677-to-0.7 origin/0.7
cd .worktree/backport-677-to-0.7
git switch --create backport-677-to-0.7
git cherry-pick -x 205f6bc2913e19442801cb78350616cf80919306

[github-actions]

Backport failed for 0.8, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin 0.8
git worktree add -d .worktree/backport-677-to-0.8 origin/0.8
cd .worktree/backport-677-to-0.8
git switch --create backport-677-to-0.8
git cherry-pick -x 205f6bc2913e19442801cb78350616cf80919306

[github-actions]

Successfully created backport PR for 8.0:

• [8.0] Solve issues with mutexes while running tests on MacOS M1  #679