Skip to content

Comments

Asyncio Client Fixes (2)#780

Open
yuce wants to merge 4 commits intohazelcast:masterfrom
yuce:asyncio-ga2
Open

Asyncio Client Fixes (2)#780
yuce wants to merge 4 commits intohazelcast:masterfrom
yuce:asyncio-ga2

Conversation

@yuce
Copy link
Contributor

@yuce yuce commented Feb 19, 2026

Some small and big fixes and improvements to mostly the asyncio client, but some asyncore as well:

  • Check the existence of event handlers in *Map proxies.
  • Cluster get_member must use the correct variable
  • Compact schema send retry, blocking cloud discovery refresh
  • Create tasks for conn.close_connection calls.
  • Fix ConnectionManager start_connect_all_members task
  • Fixed the deadlock in asyncio listener
  • Fixed the race condition in ProxyManager.get_or_create
  • Fixed the race condition in _handle_successful auth
  • Fixed wrong method retries in VC
  • Remove the write loop in reactor
  • VC put_if_absent must copy the passed document before updating it.

yuce added 3 commits February 19, 2026 16:11
* Fixed the race condition in ProxyManager.get_or_create
* Check the existence of event handlers in *Map proxies.
* VC put_if_absent must copy the passed document before updating it.
* Fix ConnectionManager start_connect_all_members task
@yuce yuce changed the title Asyncio ga2 Asyncio Client Fixes (2) Feb 19, 2026
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 87.80488% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.55%. Comparing base (57b5175) to head (e3d074f).

Files with missing lines Patch % Lines
hazelcast/internal/asyncio_proxy/map.py 55.55% 8 Missing ⚠️
hazelcast/internal/asyncio_proxy/manager.py 75.00% 4 Missing ⚠️
hazelcast/internal/asyncio_cluster.py 50.00% 1 Missing ⚠️
hazelcast/internal/asyncio_connection.py 98.11% 1 Missing ⚠️
...elcast/internal/asyncio_proxy/vector_collection.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #780      +/-   ##
==========================================
- Coverage   94.58%   94.55%   -0.04%     
==========================================
  Files         393      393              
  Lines       25083    25130      +47     
==========================================
+ Hits        23726    23761      +35     
- Misses       1357     1369      +12     

☔ 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 requested a review from emreyigit February 19, 2026 15:16
@yuce yuce mentioned this pull request Feb 20, 2026
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