Skip to content

Fix data race in lockReplication map access#1136

Merged
flimzy merged 1 commit intomainfrom
fixRace
Feb 6, 2026
Merged

Fix data race in lockReplication map access#1136
flimzy merged 1 commit intomainfrom
fixRace

Conversation

@flimzy
Copy link
Member

@flimzy flimzy commented Feb 6, 2026

The returned unlock closure was reading from the replicationMUs map without holding masterMU, racing with concurrent lockReplication calls that write to the same map.

The returned unlock closure was reading from the replicationMUs map
without holding masterMU, racing with concurrent lockReplication calls
that write to the same map.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.08%. Comparing base (6e8b308) to head (6fb9e0a).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1136      +/-   ##
==========================================
+ Coverage   62.95%   63.08%   +0.13%     
==========================================
  Files         290      290              
  Lines       19997    19998       +1     
==========================================
+ Hits        12589    12616      +27     
+ Misses       6857     6831      -26     
  Partials      551      551              

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@flimzy flimzy merged commit 33ad57f into main Feb 6, 2026
18 checks passed
@flimzy flimzy deleted the fixRace branch February 6, 2026 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant