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

Origins should be restricted to eight bytes #61

Open
olorin opened this issue Nov 1, 2014 · 1 comment
Open

Origins should be restricted to eight bytes #61

olorin opened this issue Nov 1, 2014 · 1 comment

Comments

@olorin
Copy link
Contributor

olorin commented Nov 1, 2014

  • Variable-length fields cause all sorts of problems in practice (most immediately, I can't implement fixed-width telemetry messages).
  • This doesn't limit practical applications; if you need more than 2**64 data origins in the next hundred years you're doing something very wrong.
  • Six bytes is the convention, but there's no point wasting the last two bytes of the word.

This is a breaking change, so I suggest it be slipped into v3. For now, I propose a documentation change indicating a limit-by-convention, and that origins in telemetry messages be truncated to eight bytes.

@istathar
Copy link
Owner

istathar commented Nov 2, 2014

+1

@olorin olorin added the v3 label Dec 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants