Skip to content
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

Help UDP fit wrt connection #862

Merged
merged 2 commits into from Jul 6, 2021
Merged

Help UDP fit wrt connection #862

merged 2 commits into from Jul 6, 2021

Conversation

gorryfair
Copy link
Contributor

Added a couple of sentences to Arch that are hoped to early introduce that connections are things in the API, not aretfacts of the transport protocol instances.

Different to, but related to an issue for the description in API.

Added a couple of sentences to Arch that are hoped to early introduce that connections are things in the API, not aretfacts of the transport protocol instances.
@gorryfair gorryfair requested a review from tfpauly June 14, 2021 09:55
@gorryfair gorryfair self-assigned this Jun 14, 2021
@csperkins
Copy link
Contributor

"A Connection is an abstraction that represents the communication. If the transport services interface selects a protocol such as TCP for the communication, a Connection will correspond to an underlying protocol connection. In the other hand, if a protocol such as UDP is selected, the Connection remains an abstraction in the end points and doesn't correspond to explicit protocol state" ?

Added most, but not all of what was proposed. Is this OK?
@gorryfair
Copy link
Contributor Author

@csperkins : Could you live with the update in this PR?

@csperkins
Copy link
Contributor

yes, I think so (assuming I'm reading the diff right)

@gorryfair gorryfair merged commit 39c11c2 into master Jul 6, 2021
@gorryfair gorryfair deleted the gorryfair-patch-7 branch July 6, 2021 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants