Skip to content

Conversation

@mattsu2020
Copy link
Contributor

Summary

  • Add a benchmark for sort -g (general numeric sort) using decimal and exponent notation data.
  • Focus on performance coverage for mixed numeric formats.

related

#9839

@github-actions
Copy link

github-actions bot commented Jan 7, 2026

GNU testsuite comparison:

Congrats! The gnu test tests/sort/sort-stale-thread-mem is no longer failing!
Congrats! The gnu test tests/tail/follow-name is no longer failing!
Congrats! The gnu test tests/tty/tty-eof is no longer failing!

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 7, 2026

Merging this PR will degrade performance by 13.06%

Summary

❌ 1 regressed benchmark
✅ 138 untouched benchmarks
🆕 1 new benchmark
⏩ 37 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
sort_long_line[160000] 1.7 ms 1.9 ms -13.06%
🆕 sort_general_numeric[200000] N/A 724.1 ms N/A

Comparing mattsu2020:bench-sort-general-numeric (4ca2965) with main (baf18ab)

Open in CodSpeed

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.

@github-actions
Copy link

github-actions bot commented Jan 7, 2026

GNU testsuite comparison:

Congrats! The gnu test tests/sort/sort-stale-thread-mem is no longer failing!
Congrats! The gnu test tests/tty/tty-eof is no longer failing!

@sylvestre sylvestre merged commit ecf3353 into uutils:main Jan 7, 2026
128 of 130 checks passed
@mattsu2020 mattsu2020 deleted the bench-sort-general-numeric branch January 7, 2026 11:17
sgmarz pushed a commit to sgmarz/coreutils that referenced this pull request Jan 7, 2026
---------

Co-authored-by: Sylvestre Ledru <sylvestre@debian.org>
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