-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[pkg/telemetryquerylanguage] Add TQL mapping functions for InstrumentationScope attributes #13640
[pkg/telemetryquerylanguage] Add TQL mapping functions for InstrumentationScope attributes #13640
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for keeping the TQL up-to-date with the Spec!
@pirgeo please add a changelog entry for this change |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blocking this because we need to stop adding copy paste code. Waiting for #13650
@pirgeo please rebase this. |
0c1be3a
to
928c613
Compare
Since the underlying change made this a very small change, I reorganized the commits a bit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add new tests scenarios in scope_test.go for the new getters/setters?
@pirgeo can you address the build failures? |
Co-authored-by: Tyler Helmuth <[email protected]>
3dfe554
to
38bb896
Compare
Description:
Adds mapping functions for InstrumentationLibraryScope.
Link to tracking Issue:
Resolves #13639
Documentation: Updated the respective README.md files.