-
Notifications
You must be signed in to change notification settings - Fork 35
rewrite model section #172
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
Conversation
draft-lcurley-moq-transport.md
Outdated
The application is solely responsible for the contents of objects. | ||
This includes the underlying encoding, compression, any end-to-end encryption, or authentication. | ||
A relay MUST NOT combine, split, or otherwise modify object payloads. | ||
Objects are comprised of two parts: metadata and a payload. The metadata is visible to relays but the payload is only visible to the producer and consumer. The application is solely responsible for the content of the object payload. |
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.
I would rather not loose priority and TTL in metadata here.
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.
We spent a while debating the object/group text so I'd prefer smaller changes.
@@ -167,36 +167,32 @@ x (b): | |||
: Indicates that x consists of a variable length integer, followed by that many bytes of binary data. | |||
|
|||
|
|||
# Model | |||
# Object Model {#model} |
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.
What about working top-down right than bottom up?
I think it makes more sense to describe tracks first, and then groups, and then objects, and then anything smaller (application).
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.
totally agree .. This was my suggestion on google docs too . Its more natural that way.
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.
I also had the same suggestion, though I think reading bottom up is still getting the job done. If we're pressed for time today for -05, I would leave this and revisit it later.
draft-lcurley-moq-transport.md
Outdated
The application is solely responsible for the contents of objects. | ||
This includes the underlying encoding, compression, any end-to-end encryption, or authentication. | ||
A relay MUST NOT combine, split, or otherwise modify object payloads. | ||
Objects are comprised of two parts: metadata and a payload. The metadata is visible to relays but the payload is only visible to the producer and consumer. The application is solely responsible for the content of the object payload. |
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.
payload is opaque and possibly encrypted to the relays
@@ -167,36 +167,32 @@ x (b): | |||
: Indicates that x consists of a variable length integer, followed by that many bytes of binary data. | |||
|
|||
|
|||
# Model | |||
# Object Model {#model} |
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.
totally agree .. This was my suggestion on google docs too . Its more natural that way.
No description provided.