Skip to content

Allow instrument Enabled implementation to have additional optimizations and features#4747

Merged
carlosalberto merged 5 commits intoopen-telemetry:mainfrom
pellared:improve-sdk-ins-enabled
Dec 1, 2025
Merged

Allow instrument Enabled implementation to have additional optimizations and features#4747
carlosalberto merged 5 commits intoopen-telemetry:mainfrom
pellared:improve-sdk-ins-enabled

Conversation

@pellared
Copy link
Copy Markdown
Member

@pellared pellared marked this pull request as ready for review November 21, 2025 19:42
@pellared pellared requested review from a team as code owners November 21, 2025 19:42
Copy link
Copy Markdown
Member

@jack-berg jack-berg left a comment

Choose a reason for hiding this comment

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

Confirmed this language is consistent with the corresponding language in logs and traces.

@carlosalberto carlosalberto added this pull request to the merge queue Dec 1, 2025
Merged via the queue into open-telemetry:main with commit d44e32d Dec 1, 2025
7 checks passed
@carlosalberto carlosalberto mentioned this pull request Dec 5, 2025
github-merge-queue bot pushed a commit that referenced this pull request Dec 12, 2025
### Context

- Make the W3C randomness flag required.

([#4761](#4761))

### Traces

- Deprecate Zipkin exporter document and make exporter implementation
optional.

([#4715](#4715))
- Add spec for `AlwaysRecord` sampler

([#4699](#4699))

### Metrics

- Stabilize `Enabled` API for synchronous instruments.

([#4746](#4746))
- Allow instrument `Enabled` implementation to have additional
optimizations and features.

([#4747](#4747))

### Logs

- Stabilize `LogRecordProcessor.Enabled`.

([#4717](#4717))

### SDK Configuration

- Clarifies that guidance related to boolean environment variables is
not applicable
to other configuration interfaces.
([#4723](#4723))

---------

Co-authored-by: Armin Ruech <7052238+arminru@users.noreply.github.com>
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.

7 participants