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

[metrics] Measure HistogramReporter::report #11368

Merged
merged 1 commit into from
Apr 26, 2023
Merged

[metrics] Measure HistogramReporter::report #11368

merged 1 commit into from
Apr 26, 2023

Conversation

andll
Copy link
Contributor

@andll andll commented Apr 26, 2023

We now have quite a few usages of precise histogram (and adding more). Make sense to keep an eye on how much time we spend on calculations.

@andll andll requested a review from longbowlu April 26, 2023 18:35
@andll andll enabled auto-merge (squash) April 26, 2023 18:35
@vercel
Copy link

vercel bot commented Apr 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

4 Ignored Deployments
Name Status Preview Comments Updated (UTC)
explorer ⬜️ Ignored (Inspect) Apr 26, 2023 6:37pm
explorer-storybook ⬜️ Ignored (Inspect) Apr 26, 2023 6:37pm
sui-wallet-kit ⬜️ Ignored (Inspect) Apr 26, 2023 6:37pm
wallet-adapter ⬜️ Ignored (Inspect) Apr 26, 2023 6:37pm

We now have quite a few usages of precise histogram (and adding more). Make sense to keep an eye on how much time we spend on calculations.
@andll andll requested a review from mwtian April 26, 2023 18:57
@andll andll merged commit 8793105 into main Apr 26, 2023
@andll andll deleted the andrey-182 branch April 26, 2023 19:03
longbowlu pushed a commit that referenced this pull request Apr 26, 2023
We now have quite a few usages of precise histogram (and adding more).
Make sense to keep an eye on how much time we spend on calculations.
ebmifa pushed a commit that referenced this pull request Apr 27, 2023
We now have quite a few usages of precise histogram (and adding more).
Make sense to keep an eye on how much time we spend on calculations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants