Skip to content

[HZ-5432] AtomicReference for Asyncio#815

Merged
yuce merged 11 commits into
hazelcast:masterfrom
yuce:asyncio-atomic-ref
Jun 3, 2026
Merged

[HZ-5432] AtomicReference for Asyncio#815
yuce merged 11 commits into
hazelcast:masterfrom
yuce:asyncio-atomic-ref

Conversation

@yuce
Copy link
Copy Markdown
Contributor

@yuce yuce commented Jun 2, 2026

Straightforward port of AtomicReference to Asyncio.
Depends on: #814

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 2, 2026

Codecov Report

❌ Patch coverage is 84.00000% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.37%. Comparing base (cad6111) to head (873c22b).

Files with missing lines Patch % Lines
...zelcast/internal/asyncio_proxy/atomic_reference.py 82.97% 16 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #815      +/-   ##
==========================================
- Coverage   94.40%   94.37%   -0.04%     
==========================================
  Files         407      408       +1     
  Lines       26817    26917     +100     
==========================================
+ Hits        25317    25402      +85     
- Misses       1500     1515      +15     

☔ 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.

@yuce yuce changed the title AtomicReference for Asyncio [HZ-5432] AtomicReference for Asyncio Jun 3, 2026
@yuce yuce requested a review from emreyigit June 3, 2026 09:33
The instance is created on CP Subsystem.

If no group name is given within the ``name`` argument, then the
AtomicLong instance will be created on the DEFAULT CP group.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
AtomicLong instance will be created on the DEFAULT CP group.
AtomicReference instance will be created on the DEFAULT CP group.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The asyncore doc string also has the same typo.
Fixed both at: 873c22b

@yuce yuce merged commit 2ad1551 into hazelcast:master Jun 3, 2026
11 checks passed
@yuce yuce deleted the asyncio-atomic-ref branch June 3, 2026 13:15
yuce added a commit that referenced this pull request Jun 4, 2026
Straightforward port of `CountdownLatch` for asyncio.
Depends on:
#815
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.

3 participants