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(perf): Add ability to put measurements directly on spans. #2967

Merged
merged 7 commits into from
Apr 30, 2024

Conversation

colin-sentry
Copy link
Member

This functionality is already in other SDKs, and will be used in future AI features.

@colin-sentry colin-sentry marked this pull request as ready for review April 10, 2024 20:55
@colin-sentry colin-sentry changed the title feat(performance): Add ability to put measurements directly on spans. feat(perf): Add ability to put measurements directly on spans. Apr 10, 2024
sentry_sdk/tracing.py Outdated Show resolved Hide resolved
Copy link
Member

@Zylphrex Zylphrex left a comment

Choose a reason for hiding this comment

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

LGTM, not sure if @antonpirker has other thoughts

Copy link
Member

@antonpirker antonpirker left a comment

Choose a reason for hiding this comment

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

Yes, looks OK.

But please do not merge for now. (Would make our live easier in case we need to fix something in SDK 2.0 that is discovered while dogfooding the SDK 2.0rc5 in sentry.io itself. We want to dog food until next week and then make the final 2.0 release)

Base automatically changed from sentry-sdk-2.0 to master April 25, 2024 09:13
@sentrivana
Copy link
Contributor

Sorry for the wait folks! As 2.0 is out now, we're back to business so merging this!

@sentrivana sentrivana merged commit 5130590 into master Apr 30, 2024
110 of 111 checks passed
@sentrivana sentrivana deleted the span_measurements branch April 30, 2024 08:59
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.

4 participants