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

Use octets rather than bytes throughout #1912

Closed
wants to merge 2 commits into from
Closed

Use octets rather than bytes throughout #1912

wants to merge 2 commits into from

Conversation

martinthomson
Copy link
Member

I have a feeling that Ian isn't going to like this because it messes with the notion of "bytes in flight" that we use in -recovery. I would prefer to be consistent though. For the transport, "bytes in flight" only appeared once, but its use wasn't critical.

Also closes #1863.

@martinthomson martinthomson added editorial An issue that does not affect the design of the protocol; does not require consensus. -transport labels Oct 25, 2018
@@ -618,8 +617,8 @@ sent_packets:
: An association of packet numbers to information about them, including a number
field indicating the packet number, a time field indicating the time a packet
was sent, a boolean indicating whether the packet is ack-only, a boolean
indicating whether it counts towards bytes in flight, and a bytes
field indicating the packet's size. sent_packets is ordered by packet number,
indicating whether it counts towards bytes in flight, and a size field
Copy link
Contributor

Choose a reason for hiding this comment

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

It says bytes in flight

@ianswett
Copy link
Contributor

Q: Why not standardize on bytes? All the TCP RFCs use byte, and the word octet does not occur in any TCP RFCs I can find.

@nibanks
Copy link
Member

nibanks commented Oct 25, 2018

I would also prefer bytes over octets, if we are taking a vote.

@dtikhonov
Copy link
Member

I vote for bytes.

@mikkelfj
Copy link
Contributor

bytes are not quantifiable. They could be 6-bit (as if I care).

Why not also little endian now we are at it?

@martinthomson
Copy link
Member Author

Right now, we're inconsistent. I don't believe that there is any problem along the lines @mikkelfj describes with using bytes - that ship sailed a long time ago - but HTTP/2 went with octets and we've been inconsistent. I can easily do a search and replace in the other direction; and I don't personally care.

@mikkelfj
Copy link
Contributor

then bytes it is - I have never heard of bandwidth measured in MOps

@martinthomson
Copy link
Member Author

See #1921.

@MikeBishop
Copy link
Contributor

MikeBishop commented Oct 29, 2018

@mikkelfj, see here, for example. Capacities are described in Go, Mo, etc. in French. Bandwidth is usually still in bits, though.

Bytes have historically varied in size, as large as 64 bits. The C and C++ standards define a byte as being at least eight bits in size. (See their FAQ here.) I'm very much in favor of retaining octets, because that's not a platform-specific term.

@mikkelfj
Copy link
Contributor

@MikeBishop yeah - I discovered the french connection subsequently but more often MOps means mega outputs pers second or mega operations per second or similar.

@mikkelfj
Copy link
Contributor

The french also invented the decimal system. For a short while they also had decimal time. Wonder why it never took off...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-transport editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clarify amplification limit
6 participants