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

feat(spans): Resource response sizes as measurements #2845

Merged
merged 3 commits into from
Dec 13, 2023

Conversation

jjbayer
Copy link
Member

@jjbayer jjbayer commented Dec 12, 2023

Until now, we were able to annotate span metrics and span kafka messages with arbitrary key value pairs using span.sentry_tags. But sentry_tags only supports string values, which is why span.measurements was introduced.

Copy resource response sizes into measurements so we can sort by them in the spans dataset.

#skip-changelog

@jjbayer jjbayer changed the title Feat/span measurements feat(spans): Resource response sizes as measurements Dec 12, 2023
@jjbayer jjbayer marked this pull request as ready for review December 12, 2023 15:43
@jjbayer jjbayer requested a review from a team as a code owner December 12, 2023 15:43
Copy link
Contributor

@DominikB2014 DominikB2014 left a comment

Choose a reason for hiding this comment

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

This is awesome!

@jjbayer jjbayer merged commit d8997cb into master Dec 13, 2023
23 checks passed
@jjbayer jjbayer deleted the feat/span-measurements branch December 13, 2023 10:37
jan-auer added a commit that referenced this pull request Dec 19, 2023
* master: (35 commits)
  fix(spans): Parse quotes in MySQL (#2846)
  ref(cardinality): Use a Lua script and in-memory cache for the cardinality limiter (#2849)
  fix(spans): Detect hex with fallback scrubber (#2868)
  release: 23.12.0
  Revert "ci: Update upload-artifact and download-artifact actions" (#2866)
  Revert "build: Update axum and http" (#2863)
  feat(spans): Allow resource.img spans (#2855)
  build: Update axum and http (#2844)
  fix(build): Add additional dependencies to the release build (#2858)
  ci: Update upload-artifact and download-artifact actions (#2861)
  feat(spans): Parse timestamps from strings (#2857)
  fix(spans): Scrub integer file extensions (#2856)
  feat(spans): Remove unused transaction tag from resource metrics (#2853)
  ref(cardinality): Recover buckets on cardinality limiter failure (#2852)
  feat(server): Org rate limit per metric bucket (#2836)
  ref(spans): List metric tags explicitly (#2834)
  feat(spans): Resource response sizes as measurements (#2845)
  feat(crons): Add thresholds to monitor config payload (#2842)
  feat(spans): Allow ingestion of metrics summary on spans (#2823)
  ref(crons): Add documentation to CheckInMessageType (#2840)
  ...
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.

3 participants