Skip to content

CICD.yml: Drop checks for hashsum#10259

Merged
RenjiSann merged 1 commit intouutils:mainfrom
oech3:patch-9
Jan 16, 2026
Merged

CICD.yml: Drop checks for hashsum#10259
RenjiSann merged 1 commit intouutils:mainfrom
oech3:patch-9

Conversation

@oech3
Copy link
Contributor

@oech3 oech3 commented Jan 15, 2026

@RenjiSann Not conflict with your PR, maybe.

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 15, 2026

Merging this PR will degrade performance by 27.39%

⚡ 1 improved benchmark
❌ 8 regressed benchmarks
✅ 273 untouched benchmarks
⏩ 38 skipped benchmarks1

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

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Memory sort_numeric[500000] 75.5 MB 79.2 MB -4.67%
Memory sort_key_field[500000] 47.8 MB 51.8 MB -7.62%
Memory sort_unique_locale[500000] 33.6 MB 39.8 MB -15.51%
Memory sort_ascii_utf8_locale 6.7 MB 6.2 MB +7.93%
Memory sort_accented_data[500000] 22.1 MB 28.3 MB -21.79%
Memory sort_mixed_data[500000] 22.9 MB 27.1 MB -15.71%
Memory sort_ascii_only[500000] 22.2 MB 28.3 MB -21.77%
Memory sort_long_line[160000] 712.6 KB 981.4 KB -27.39%
Memory du_wide_tree[(5000, 500)] 1.2 MB 1.2 MB -3.36%

Comparing oech3:patch-9 (df1e043) with main (2c75e71)

Open in CodSpeed

Footnotes

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

@oech3

This comment was marked as resolved.

@RenjiSann
Copy link
Collaborator

Thanks !

@RenjiSann RenjiSann merged commit fe748f4 into uutils:main Jan 16, 2026
156 of 157 checks passed
@oech3 oech3 deleted the patch-9 branch January 16, 2026 13:41
@oech3
Copy link
Contributor Author

oech3 commented Jan 16, 2026

I think we can split PR for test_hashsum.rs cleanup at least for --md5, etc abd hashsum standalone.

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