Skip to content

Commit

Permalink
Merge pull request #16257 from alexzhang1030/calc_deterministic_verbose
Browse files Browse the repository at this point in the history
perf: remove assignDeterministic verbose Math.ceil code to improve performance
  • Loading branch information
sokra committed Nov 9, 2022
2 parents f7f36ad + 61dee6e commit 6f2c5e8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/ids/IdHelpers.js
Expand Up @@ -387,7 +387,7 @@ const assignDeterministicIds = (

// max 5% fill rate
const optimalRange = Math.min(
Math.ceil(items.length * 20) + extraSpace,
items.length * 20 + extraSpace,
Number.MAX_SAFE_INTEGER
);

Expand Down

0 comments on commit 6f2c5e8

Please sign in to comment.