Skip to content

fix(wasi): sort ordering check and external-merge separator - #13688

Open
eduardomourar wants to merge 1 commit into
uutils:mainfrom
eduardomourar:wasi-sort-fixes
Open

fix(wasi): sort ordering check and external-merge separator#13688
eduardomourar wants to merge 1 commit into
uutils:mainfrom
eduardomourar:wasi-sort-fixes

Conversation

@eduardomourar

Copy link
Copy Markdown
Contributor

sort's ordering check runs single-threaded on WASI, which has no thread support, reading every chunk up front instead of streaming from a background reader thread. Its external-merge path also inserts a missing separator between concatenated files so lines don't merge across file boundaries.

@codspeed-hq

codspeed-hq Bot commented Aug 1, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 345 untouched benchmarks
⏩ 46 skipped benchmarks1


Comparing eduardomourar:wasi-sort-fixes (62496f4) with main (0c8a3c7)

Open in CodSpeed

Footnotes

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

sort's ordering check runs single-threaded on WASI, which has no
thread support, reading every chunk up front instead of streaming
from a background reader thread. Its external-merge path also inserts
a missing separator between concatenated files so lines don't merge
across file boundaries.
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.

1 participant