Skip to content

fix: benchmark panic issue resolved.#3538

Draft
Jyothish-POC wants to merge 1 commit into
open-telemetry:mainfrom
Jyothish-POC:benchmark_panic
Draft

fix: benchmark panic issue resolved.#3538
Jyothish-POC wants to merge 1 commit into
open-telemetry:mainfrom
Jyothish-POC:benchmark_panic

Conversation

@Jyothish-POC

@Jyothish-POC Jyothish-POC commented Jul 21, 2026

Copy link
Copy Markdown

Change Summary

Running cargo bench -p benchmarks --bench exporter from otap-dataflow directory panics on the first iteration:

What issue does this PR close?

How are these changes tested?

While running cargo bench -p benchmarks --bench exporter now I am getting the below result.

image

Are there any user-facing changes?

Changelog

  • Added a .chloggen/*.yaml entry
  • This PR is a chore (indicated in title)
  • This is a documentation-only PR.

@linux-foundation-easycla

Copy link
Copy Markdown

CLA Missing ID

  • ❌ The email address for the commit (83db462) is not linked to the GitHub account, preventing the EasyCLA check. Consult this Help Article and GitHub Help to resolve. (To view the commit's email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please visit our EasyCLA portal and chat with our support bot.

@github-actions github-actions Bot added the lang:rust Pull requests that update Rust code label Jul 21, 2026
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 21, 2026

Copy link
Copy Markdown

Pull request dashboard status

Status last refreshed: 2026-07-23 05:26:11 UTC.

  • Waiting on: Author
  • Next step: Move out of draft to request review.

This automated status or its linked feedback items may be incorrect. If something looks wrong, please report it with the result you expected.

@Jyothish-POC

Copy link
Copy Markdown
Author

Trying to develop the enhancement idea thats mentioned on the issue.

Optional follow-up:

create_otap_batch produces a single-row batch, so the bench's row_size knob loses meaning. A small enhancement (also welcome from a new contributor) is to add create_otap_batch_with_rows(batch_id, payload_type, num_rows) next to create_otap_batch in otap_mock.rs and have the bench vary num_rows so each message carries varying payload weight.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang:rust Pull requests that update Rust code

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

[benchmarks/exporter] benchmark panic

1 participant