Skip to content

benches: adjust sort_long_line line size to 10,000 bytes#11057

Merged
sylvestre merged 1 commit intouutils:mainfrom
xtqqczze:sort_long_line
Feb 22, 2026
Merged

benches: adjust sort_long_line line size to 10,000 bytes#11057
sylvestre merged 1 commit intouutils:mainfrom
xtqqczze:sort_long_line

Conversation

@xtqqczze
Copy link
Contributor

@xtqqczze xtqqczze commented Feb 22, 2026

Adjust line size to a nice round 10,000 (which isn't a multiple of START_BUFFER_SIZE (8000) making it a more realistic benchmark).

Follow-up #11033

@xtqqczze xtqqczze changed the title benches: adjust sort_long_line line_size to 10,000 bytes benches: adjust sort_long_line line size to 10,000 bytes Feb 22, 2026
@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/pr/bounded-memory (passes in this run but fails in the 'main' branch)

@xtqqczze
Copy link
Contributor Author

Benchmark results show sort_long_line[10000] using 9.1% more memory than sort_long_line[16000] supporting the idea it is a more realistic benchmark.

@sylvestre sylvestre merged commit 1d113b5 into uutils:main Feb 22, 2026
157 checks passed
@xtqqczze xtqqczze deleted the sort_long_line branch February 22, 2026 19:17
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