Skip to content

[exporter/exporterhelper] mark exporter.PersistRequestContext as stable#15330

Merged
codeboten merged 3 commits into
open-telemetry:mainfrom
codeboten:codeboten/stabilize-exporter.PersistRequestContext
May 21, 2026
Merged

[exporter/exporterhelper] mark exporter.PersistRequestContext as stable#15330
codeboten merged 3 commits into
open-telemetry:mainfrom
codeboten:codeboten/stabilize-exporter.PersistRequestContext

Conversation

@codeboten

Copy link
Copy Markdown
Contributor

This feature gate has been in beta for 15 versions, it's time to stabilize it.

This feature gate has been in beta for 15 versions, it's time to stabilize it.

Signed-off-by: Alex Boten <[email protected]>
@codeboten
codeboten requested review from a team, bogdandrutu and dmitryax as code owners May 20, 2026 20:32
Signed-off-by: Alex Boten <[email protected]>
@codspeed-hq

codspeed-hq Bot commented May 20, 2026

Copy link
Copy Markdown

Merging this PR will improve performance by 27.66%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

⚡ 1 improved benchmark
✅ 6 untouched benchmarks
⏩ 76 skipped benchmarks1

Performance Changes

Benchmark BASE HEAD Efficiency
BenchmarkTraceSizeSpanCount 60 ns 47 ns +27.66%

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing codeboten:codeboten/stabilize-exporter.PersistRequestContext (c7dc4d9) with main (58cb3e5)

Open in CodSpeed

Footnotes

  1. 76 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@codecov

codecov Bot commented May 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.24%. Comparing base (58cb3e5) to head (c7dc4d9).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #15330      +/-   ##
==========================================
+ Coverage   91.23%   91.24%   +0.01%     
==========================================
  Files         706      706              
  Lines       46165    46165              
==========================================
+ Hits        42118    42124       +6     
+ Misses       2836     2832       -4     
+ Partials     1211     1209       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@codeboten
codeboten enabled auto-merge May 20, 2026 20:57
@codeboten
codeboten added this pull request to the merge queue May 21, 2026
Merged via the queue into open-telemetry:main with commit 0fca28a May 21, 2026
67 checks passed
@codeboten
codeboten deleted the codeboten/stabilize-exporter.PersistRequestContext branch May 21, 2026 14:29
pull Bot pushed a commit to weiyilai/opentelemetry-collector that referenced this pull request May 21, 2026
TimoBehrendt pushed a commit to TimoBehrendt/tracebasedlogsampler that referenced this pull request Jun 1, 2026
This PR contains the following updates:

| Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) |
|---|---|---|---|
| [go.opentelemetry.io/collector/component](https://github.com/open-telemetry/opentelemetry-collector) | `v1.58.0` → `v1.59.0` | ![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2fcomponent/v1.59.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2fcomponent/v1.58.0/v1.59.0?slim=true) |
| [go.opentelemetry.io/collector/component/componenttest](https://github.com/open-telemetry/opentelemetry-collector) | `v0.152.0` → `v0.153.0` | ![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2fcomponent%2fcomponenttest/v0.153.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2fcomponent%2fcomponenttest/v0.152.0/v0.153.0?slim=true) |
| [go.opentelemetry.io/collector/confmap](https://github.com/open-telemetry/opentelemetry-collector) | `v1.58.0` → `v1.59.0` | ![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2fconfmap/v1.59.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2fconfmap/v1.58.0/v1.59.0?slim=true) |
| [go.opentelemetry.io/collector/consumer](https://github.com/open-telemetry/opentelemetry-collector) | `v1.58.0` → `v1.59.0` | ![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2fconsumer/v1.59.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2fconsumer/v1.58.0/v1.59.0?slim=true) |
| [go.opentelemetry.io/collector/consumer/consumertest](https://github.com/open-telemetry/opentelemetry-collector) | `v0.152.0` → `v0.153.0` | ![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2fconsumer%2fconsumertest/v0.153.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2fconsumer%2fconsumertest/v0.152.0/v0.153.0?slim=true) |
| [go.opentelemetry.io/collector/pdata](https://github.com/open-telemetry/opentelemetry-collector) | `v1.58.0` → `v1.59.0` | ![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2fpdata/v1.59.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2fpdata/v1.58.0/v1.59.0?slim=true) |
| [go.opentelemetry.io/collector/processor](https://github.com/open-telemetry/opentelemetry-collector) | `v1.58.0` → `v1.59.0` | ![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2fprocessor/v1.59.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2fprocessor/v1.58.0/v1.59.0?slim=true) |
| [go.opentelemetry.io/collector/processor/processortest](https://github.com/open-telemetry/opentelemetry-collector) | `v0.152.0` → `v0.153.0` | ![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2fprocessor%2fprocessortest/v0.153.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2fprocessor%2fprocessortest/v0.152.0/v0.153.0?slim=true) |

---

### Release Notes

<details>
<summary>open-telemetry/opentelemetry-collector (go.opentelemetry.io/collector/component)</summary>

### [`v1.59.0`](https://github.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v1590v01530)

##### 🛑 Breaking changes 🛑

- `pkg/configoptional`: Stabilize feature gate configoptional.AddEnabledField ([#&#8203;15333](open-telemetry/opentelemetry-collector#15333))
- `pkg/confmap`: Stabilize confmap.newExpandedValueSanitizer feature gate ([#&#8203;15339](open-telemetry/opentelemetry-collector#15339))
- `pkg/exporterhelper`: mark exporter.PersistRequestContext as stable ([#&#8203;15330](open-telemetry/opentelemetry-collector#15330))
- `pkg/otelcol`: Stabilize otelcol.printInitialConfig gate ([#&#8203;15340](open-telemetry/opentelemetry-collector#15340))
- `pkg/pdata`: Remove pdata.useCustomProtoEncoding feature gate ([#&#8203;15332](open-telemetry/opentelemetry-collector#15332))
- `pkg/service`: Stabilize telemetry.UseLocalHostAsDefaultMetricsAddress gate ([#&#8203;15342](open-telemetry/opentelemetry-collector#15342))
- `pkg/xpdata`: Stabilize pdata.enableRefCounting feature gate ([#&#8203;15331](open-telemetry/opentelemetry-collector#15331))

##### 🧰 Bug fixes 🧰

- `pkg/config/configgrpc`: Fix memory corruption and fatal error in Snappy ([#&#8203;15237](open-telemetry/opentelemetry-collector#15237), [#&#8203;15320](open-telemetry/opentelemetry-collector#15320))

<!-- previous-version -->

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://github.com/renovatebot/renovate/discussions) if that's undesired.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My41LjQiLCJ1cGRhdGVkSW5WZXIiOiI0My41LjQiLCJ0YXJnZXRCcmFuY2giOiJtYWluIiwibGFiZWxzIjpbXX0=-->

Reviewed-on: https://gitea.t000-n.de/t.behrendt/tracebasedlogsampler/pulls/73
Reviewed-by: t.behrendt <[email protected]>
Co-authored-by: Renovate Bot <[email protected]>
Co-committed-by: Renovate Bot <[email protected]>
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.

2 participants