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.

Sign up for GitHub

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

Jump to bottom

follow-up: data type for Offsets #2782

Open

davidbeesley opened this issue Nov 7, 2022 · 0 comments

Labels

Contributor

davidbeesley commented Nov 7, 2022

Follow up to discussion on #2766
Two concerns about current treatment of offsets.

Sometimes its a u32. That gives a max-value of 4 billion, which is equivalent to 135 messages per second for one year, or 20 messages per second for five years.
It's inconsistent in general and there is a lot of conversion from one type to another and signed to unsigned.

sehz added the technical debt label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment