Don't use cached relpath in parallel code

[FayazRahman]

🚀 🚀 Pull Request

Impact

• Bug fix (non-breaking change which fixes expected existing functionality)
• Enhancement/New feature (adds functionality without impacting existing logic)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

• The custom cached relpath function is not thread-safe so don't use it in parallel code paths.

Things to be aware of

Things to worry about

Additional Context

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (d952ab6) 84.51% compared to head (20b3345) 83.92%.
Report is 3 commits behind head on main.

Files	Patch %	Lines
deeplake/core/vectorstore/deeplake_vectorstore.py	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2738      +/-   ##
==========================================
- Coverage   84.51%   83.92%   -0.59%     
==========================================
  Files         233      233              
  Lines       26452    26452              
==========================================
- Hits        22356    22200     -156     
- Misses       4096     4252     +156     

Flag	Coverage Δ
unittests	83.92% <83.33%> (-0.59%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud