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

Draft evaluation for QUIC. #17

Merged
merged 9 commits into from Jan 28, 2017
Merged

Conversation

mfoltzgoogle
Copy link
Contributor

Addresses Issue #9: Evaluate QUIC as a protocol transport

Based on conversation with a QUIC developer in Chromium and evaluation of the spec, this is a draft evaluation of QUIC using the template in PR #16.

Many details need to be determined and also reconciled with the current draft specification; my initial review was based on the public documents on chromium.org, and not the current IETF draft.

So consider this a first draft :)

# QUIC

[QUIC](https://www.chromium.org/quic) is a multiplexed stream transport over
UDP. It advertises the following benefits:
Copy link
Member

Choose a reason for hiding this comment

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

I suggest changing "benefits" to "benefits over TCP"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

controller and display. The following mapping could be used to associate
streams with Presentation API concepts:

- One reserved stream 5 for control messages between the controller and display.
Copy link
Member

Choose a reason for hiding this comment

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

"with ID 5" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@mfoltzgoogle mfoltzgoogle merged commit 2e38856 into issue-8-transport-template Jan 28, 2017
@mfoltzgoogle mfoltzgoogle deleted the issue-9-quic branch January 28, 2017 00:37
@mfoltzgoogle
Copy link
Contributor Author

Thanks for looking it over @chrisn. Merged

@mfoltzgoogle mfoltzgoogle restored the issue-9-quic branch January 28, 2017 00:40
@mfoltzgoogle mfoltzgoogle deleted the issue-9-quic branch January 28, 2017 00:42
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.

None yet

2 participants