Skip to content

Fix benchmark data#8113

Merged
jack-berg merged 3 commits intoopen-telemetry:benchmarksfrom
jack-berg:fix-benchmark-data
Feb 24, 2026
Merged

Fix benchmark data#8113
jack-berg merged 3 commits intoopen-telemetry:benchmarksfrom
jack-berg:fix-benchmark-data

Conversation

@jack-berg
Copy link
Copy Markdown
Member

When I submitted #8018, I added a trailing ; which broke the benchmark action's JSON parsing and caused it to reset the history.

I managed to find the specific log in the build history which shows this error:

Could not find data.js at benchmarks/data.js. Using empty default: SyntaxError: Unexpected non-whitespace character after JSON at position 1056628

https://github.com/open-telemetry/opentelemetry-java/actions/runs/22166116617/job/64093622854#step:11:38

This PR restores the history and removed the trailing ;.

@jack-berg jack-berg merged commit db8f6ce into open-telemetry:benchmarks Feb 24, 2026
1 check passed
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