Skip to content

[HZ-5434] FencedLock for Asyncio#818

Open
yuce wants to merge 23 commits into
hazelcast:masterfrom
yuce:asyncio-fenced-lock
Open

[HZ-5434] FencedLock for Asyncio#818
yuce wants to merge 23 commits into
hazelcast:masterfrom
yuce:asyncio-fenced-lock

Conversation

@yuce
Copy link
Copy Markdown
Contributor

@yuce yuce commented Jun 8, 2026

Ported FencedLock proxy to asyncio.

The asyncore variant uses actual thread IDs to disable re-entrancy.
That's not useful with the asyncio variant, so currently the thread ID 0 is used when necessary.
I've omitted docs that refer to threads, and tests that use threads.
I'll push another PR to add those after lock context implementation.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.18391% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.17%. Comparing base (c8d3629) to head (34dfdf7).

Files with missing lines Patch % Lines
hazelcast/internal/asyncio_proxy/fenced_lock.py 80.64% 30 Missing ⚠️
hazelcast/internal/asyncio_proxy/cp_manager.py 94.44% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #818      +/-   ##
==========================================
- Coverage   94.21%   94.17%   -0.05%     
==========================================
  Files         410      411       +1     
  Lines       27318    27492     +174     
==========================================
+ Hits        25739    25890     +151     
- Misses       1579     1602      +23     

☔ View full report in Codecov by Harness.
📢 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.

@yuce yuce requested a review from emreyigit June 8, 2026 09:04
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.

2 participants