Skip to content

join: add benchmarks#10005

Merged
sylvestre merged 1 commit into
uutils:mainfrom
WaterWhisperer:join-benchmark
Jan 3, 2026
Merged

join: add benchmarks#10005
sylvestre merged 1 commit into
uutils:mainfrom
WaterWhisperer:join-benchmark

Conversation

@WaterWhisperer

Copy link
Copy Markdown
Contributor

Supplement to #9982

@sylvestre

Copy link
Copy Markdown
Contributor

i wrote: #9991

Comment thread src/uu/join/benches/join_bench.rs Outdated
@WaterWhisperer

Copy link
Copy Markdown
Contributor Author

i wrote: #9991

Oh, sorry I didn't notice. I'll close it.

@sylvestre

Copy link
Copy Markdown
Contributor

well, we can keep yours :)

@sylvestre sylvestre reopened this Jan 3, 2026
@sylvestre

Copy link
Copy Markdown
Contributor

just add it to github action and just keep one value for the bench (and move it into the function)

@WaterWhisperer

Copy link
Copy Markdown
Contributor Author

thanks for the review

@codspeed-hq

codspeed-hq Bot commented Jan 3, 2026

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #10005 will not alter performance

Comparing WaterWhisperer:join-benchmark (54f2354) with main (ea64612)

Summary

✅ 135 untouched
🆕 3 new
⏩ 37 skipped1

Benchmarks breakdown

Benchmark BASE HEAD Efficiency
🆕 join_partial_overlap N/A 10.5 ms N/A
🆕 join_custom_separator N/A 12.1 ms N/A
🆕 join_full_match N/A 12.8 ms N/A

Footnotes

  1. 37 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.

@sylvestre

Copy link
Copy Markdown
Contributor

please keep 3 benchmarks, we probably don't need 6 as they take a long time to run

@WaterWhisperer

Copy link
Copy Markdown
Contributor Author

please keep 3 benchmarks, we probably don't need 6 as they take a long time to run

ok

@sylvestre
sylvestre merged commit e4d35b0 into uutils:main Jan 3, 2026
130 checks passed
@sylvestre

Copy link
Copy Markdown
Contributor

oh, please create a new bench marking (same file) with the French locale to compare with the current other pr
thanks

@WaterWhisperer

Copy link
Copy Markdown
Contributor Author

oh, please create a new bench marking (same file) with the French locale to compare with the current other pr thanks

do it in another PR?

@sylvestre

Copy link
Copy Markdown
Contributor

Yes please

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