Skip to content

feat(discover): Allows insight metrics to be used with the avg function in the discover metrics dataset #75870

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

Merged

Conversation

edwardgou-sentry
Copy link
Contributor

Allows insights metrics (eg: span.self_time) to be used in the metrics avg function

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 8, 2024
Copy link

codecov bot commented Aug 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #75870       +/-   ##
===========================================
- Coverage   90.49%   78.27%   -12.22%     
===========================================
  Files        3099     6824     +3725     
  Lines      193934   303942   +110008     
  Branches        0    52287    +52287     
===========================================
+ Hits       175504   237918    +62414     
- Misses      18430    59635    +41205     
- Partials        0     6389     +6389     

@edwardgou-sentry edwardgou-sentry requested a review from a team August 8, 2024 23:37
@edwardgou-sentry edwardgou-sentry marked this pull request as ready for review August 8, 2024 23:37
@edwardgou-sentry edwardgou-sentry requested a review from a team as a code owner August 8, 2024 23:37
…er' of github.com:getsentry/sentry into egou/feat/discover-metrics-avg-allow-insight-metrics
Base automatically changed from egou/feat/insights-entity-subscription-skip-metrics-layer to master August 12, 2024 15:15
@edwardgou-sentry edwardgou-sentry requested a review from a team as a code owner August 12, 2024 15:15
@edwardgou-sentry edwardgou-sentry merged commit ef2f329 into master Aug 12, 2024
50 checks passed
@edwardgou-sentry edwardgou-sentry deleted the egou/feat/discover-metrics-avg-allow-insight-metrics branch August 12, 2024 19:15
@github-actions github-actions bot locked and limited conversation to collaborators Aug 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants