-
-
Notifications
You must be signed in to change notification settings - Fork 306
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
fix: use bigint to calculate max builder boost factor #6275
Conversation
I was also reviewing the validation as mentioned in #6236 (comment) to prevent non integer values and negative numbers but as @g11tech highlighted in the standup we lose precision on the value if we use uint schema instead of string because of the same problem that it will be converted to a number which does not support 64 bit unsigned integers (only up to 2^53−1). Based on some testing I did and reviewing the usage of builder boost factor the current validation seems to be sufficient
We could further investigate how to support bigint more broadly at serialization/deserilization and schema validation level if we see the need for it and there are more values that require to support 64 bit unsigned integers. |
Performance Report✔️ no performance regression detected Full benchmark results
|
🎉 This PR is included in v1.14.0 🎉 |
Motivation
Current value of
MAX_BUILDER_BOOST_FACTOR
is off by one due to usage of number instead of bigint in the calculation.vs.
The correct value should be
18446744073709551615
(2**64 - 1)Description
Use bigint to calculate max builder boost factor