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

decorate check dispatch spans with caching and singleflight attributes #1678

Merged
merged 2 commits into from
Dec 11, 2023

Conversation

vroldanbet
Copy link
Contributor

When folks are analyzing a trace, some dispatch events are not expanded and that's typically because either it was cached or singleflighted, but there are no means to know that right now.

this adds attributes when a dispatch is cached or singleflighted

@vroldanbet vroldanbet requested a review from a team December 6, 2023 17:57
@github-actions github-actions bot added area/dependencies Affects dependencies area/dispatch Affects dispatching of requests labels Dec 6, 2023
Copy link
Member

@jzelinskie jzelinskie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after a rebase

@vroldanbet vroldanbet self-assigned this Dec 11, 2023
@vroldanbet vroldanbet enabled auto-merge December 11, 2023 12:59
@vroldanbet vroldanbet added this pull request to the merge queue Dec 11, 2023
Merged via the queue into main with commit 01c462b Dec 11, 2023
20 checks passed
@vroldanbet vroldanbet deleted the decorate-spans branch December 11, 2023 13:16
@github-actions github-actions bot locked and limited conversation to collaborators Dec 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/dependencies Affects dependencies area/dispatch Affects dispatching of requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants