-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
chore: avoid using bigint for metrics #5926
Conversation
Performance Report✔️ no performance regression detected Full benchmark results
|
@tuyennhv do you have any evidence that this usage is a problem? This allocates relatively few items per second |
@dapplion yes the usage is not much, I deployed this branch to a group and it does not show a clear difference. |
12b6772
to
f28fb1a
Compare
🎉 This PR is included in v1.12.0 🎉 |
Motivation
We don't want to use bigint if we can since it causes us memory
Description
process.hrtime()
)part of #5892