Skip to content

Conversation

@alanprot
Copy link
Member

What this PR does:
Create span to measure how long QF takes to merge responses as this method can take considerable amount of time for big response.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • [NA] Tests updated
  • [NA] Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Signed-off-by: Alan Protasio <[email protected]>
@pull-request-size pull-request-size bot added size/M and removed size/S labels Dec 30, 2022
Copy link
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

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

Thanks!

@yeya24 yeya24 merged commit dfe3737 into cortexproject:master Dec 30, 2022
alexqyle pushed a commit to alexqyle/cortex that referenced this pull request May 2, 2023
* Create Span for the codec MergeResponse method

Signed-off-by: Alan Protasio <[email protected]>

* Changelog

Signed-off-by: Alan Protasio <[email protected]>

Signed-off-by: Alan Protasio <[email protected]>
Signed-off-by: Alex Le <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants