-
Notifications
You must be signed in to change notification settings - Fork 73
Asyncio Module Ported more tests [4/6] #750
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
yuce
merged 66 commits into
hazelcast:master
from
yuce:asyncio-module-integration-tests2
Dec 4, 2025
Merged
Changes from all commits
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
cbe98dc
Initial asyncio commit
yuce 162fd17
Updates
yuce 9931956
Updates
yuce 856e3df
Merge branch 'master' into asyncio-module
yuce 35384bf
Black
yuce fdda120
Updates
yuce fee5b45
Updates
yuce fc2c38b
Updates
yuce 1772031
Removed docs, include HazelcastClient/Map in public API
yuce 170cf89
Updates
yuce 539c904
Merge branch 'master' into asyncio-module
yuce 22449a8
black
yuce 5406bc6
Ignore incorrect mypy errors
yuce a417a4a
Updates
yuce d00c480
Updates
yuce baa3bc1
Annotate optional params
yuce ebfc9e2
black
yuce 6928837
Remove update to test util
yuce 3e03cbf
black
yuce 51ced7a
black
yuce e635b94
update
yuce 4f103f6
Added support for SSL
yuce 042cc58
Added SSL tests
yuce 265a2b4
Added mutual authentication test
yuce 293975d
Added hostname verification tests
yuce 2718478
black
yuce 58783dc
Ported more integration tests
yuce 3cf9982
Ported hazelcast json value test
yuce 7e97ec7
Merge branch 'master' into asyncio-module-integration-tests1
yuce 6a558e8
Merge branch 'master' into asyncio-module-ssl
yuce a630706
Merge branch 'master' into asyncio-module
yuce c1798ea
Ported heart beat test
yuce e92936a
Ported more tests
yuce 6ced889
Merge branch 'master' into asyncio-module
yuce c313bfa
Merge branch 'master' into asyncio-module-ssl
yuce 6222c6b
Merge branch 'master' into asyncio-module-integration-tests1
yuce 120a58a
black
yuce 80880b8
Fixed type hints
yuce 6431acc
type hints
yuce e9a9b5e
Ported more tests
yuce 5334cd1
Added near cache, statistics, statistics tests
yuce a14290a
Black
yuce 492ccc1
Fixed getting local address
yuce e8a2600
Fixed getting local address, take 2
yuce 24eb6bf
Added nearcache tests
yuce 6ab9365
Ported missing nearcache test
yuce 91bf1d1
Addressed review comment
yuce 2128f5e
Removed unnecessary code
yuce 62697e3
Add BETA warning
yuce a87a5c6
Black
yuce 8d7eede
Merge branch 'asyncio-module' into asyncio-module-ssl
yuce 00a2d12
Merge branch 'asyncio-module-ssl' into asyncio-module-integration-tests1
yuce 539466b
Merge branch 'asyncio-module-integration-tests1' into asyncio-module-…
yuce e673679
Updated test_heartbeat_stopped_and_restored
yuce ab4a746
Merge branch 'asyncio-module-integration-tests1' into asyncio-module-…
yuce 319bb35
Fixed tests
yuce 8e325ea
Linter
yuce eed53b3
Test updates
yuce 1ca7fd6
Addressed review comment
yuce d9acede
updates
yuce bd23f41
Merge branch 'asyncio-module-ssl' into asyncio-module-integration-tests1
yuce 76759ec
Merge branch 'asyncio-module-integration-tests1' into asyncio-module-…
yuce 2a2d6e8
Merge branch 'master' into asyncio-module-integration-tests1
yuce 6da4226
Merge branch 'asyncio-module-integration-tests1' into asyncio-module-…
yuce 550a006
Merge branch 'master' into asyncio-module-integration-tests2
yuce 2c8f10e
Prevent deadlock
yuce File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not use simpler (really need concurrency here?):
Reads a little odd with the
returnthere -- is that even safe?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's safe, the task group will not exit without all tasks are completed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks odd to me to have the
returnthere, the following seems clearer IMO as it doesn't mix the task group completion semantics with some nested return.