Skip to content

Conversation

@MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Sep 8, 2025

Avoid allocating instrument creation option if possible. If a user does not provide any options, use a static, read-only, file-level defined option slice. Otherwise, append to the user allocated slice the description and unit options.

Avoid allocating instrument creation option if possible. If a user does
not provide any options, use a static, read-only, file-level defined
option slice. Otherwise, append to the user allocated slice the
description and unit options.
@MrAlias MrAlias added this to the v1.39.0 milestone Sep 8, 2025
@MrAlias MrAlias added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Sep 8, 2025
@MrAlias MrAlias marked this pull request as ready for review September 8, 2025 19:18
@codecov
Copy link

codecov bot commented Sep 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.9%. Comparing base (9b6585a) to head (3d159d3).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #7328   +/-   ##
=====================================
  Coverage   82.9%   82.9%           
=====================================
  Files        267     267           
  Lines      24997   24997           
=====================================
+ Hits       20743   20744    +1     
+ Misses      3878    3877    -1     
  Partials     376     376           

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@flc1125

This comment was marked as spam.

@MrAlias MrAlias merged commit 3baabce into open-telemetry:main Sep 9, 2025
29 of 30 checks passed
@MrAlias MrAlias deleted the rm-semconv-inst-opt-alloc branch September 9, 2025 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants