-
-
Notifications
You must be signed in to change notification settings - Fork 718
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
Improve differentiation between incoming/outgoing connections and transfers #6933
Merged
crusaderky
merged 23 commits into
dask:main
from
hendrikmakait:naming-for-incoming-outgoing
Aug 30, 2022
Merged
Changes from 20 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
bbe7e18
Improve naming to differentiate between incoming/outgoing connections…
hendrikmakait 485e72f
Typo
hendrikmakait cc83764
Add DeprecationWarnings
hendrikmakait f70cceb
Add test for deprecation warning
hendrikmakait 074707d
Review comments
hendrikmakait 2bfdc2e
Improve docs
hendrikmakait aea0210
Rename comm logs
hendrikmakait 0742765
Rename total connections
hendrikmakait 4b073aa
comm -> transfer
hendrikmakait 87599ed
Update docs
hendrikmakait 3afc878
Move transfer_incoming_count_total
hendrikmakait 4fb8b7b
Adjust test
hendrikmakait 1b2034d
Add deprecated property
hendrikmakait 4218493
Fix naming
hendrikmakait 07c96db
Fix tests
hendrikmakait ce1d051
Minor
hendrikmakait abdc243
Fix test
hendrikmakait d28ff03
Rename test
hendrikmakait 5d64743
rename target_message_size
hendrikmakait a0faa5f
Minor
hendrikmakait bdfc87c
Update distributed/utils_test.py
crusaderky 1f3dc4f
Update distributed/utils_test.py
crusaderky b10985a
Update distributed/worker_state_machine.py
crusaderky 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 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 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 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 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 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 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 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.
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.
For now, I don't want to touch
"Connections"
.