-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Add missing queue measurement instrumentation #6480
Conversation
konstantin-msft
commented
Sep 14, 2023
•
edited
edited
- Add missing queue measurement instrumentation.
- Default queue time to 0 for manually completed queue events.
- Update unit tests.
9e3e17b
to
76be051
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any way we can identify when addQueueMeasurement
is not called when it is supposed to?
We can do that by checking the number of manually completed queue events, which is, unfortunately, non-proactive. |
78fee29
to
a914427
Compare
- Add `performanceClient` to `BrowserCacheManager` constructor. - Default queue time to 0 for manually completed queue events. - Update unit tests.
a914427
to
d490ed6
Compare