cksum: symlink to md5sum,b2sum, and sha*sum#9984
Conversation
CodSpeed Performance ReportMerging #9984 will improve performance by ×8,800Comparing Summary
Benchmarks breakdown
Footnotes
|
a79d274 to
a132677
Compare
|
I considered that CodSpeed was broken, but it seems reproducible... |
|
|
|
@naoNao89 Any idea about CodSpeed breakage? |
|
We should fix or remove #10002 before discarding |
|
The file creation happens outside the benchmark loop, so the benchmarks are measuring almost nothing. uu/cksum/benches/cksum_bench.rs:14-24 fn $algo_name(bencher: Bencher) {
let data = text_data::generate_by_size(100, 80);
let file_path = setup_test_file(&data); // File created ONCE, outside loop
bencher.bench(|| {
black_box(run_util_function(
uumain,
&["--algorithm", $algo_str, file_path.to_str().unwrap()],
));
}); // This loop only measures reading a cached file
}I think we should:
|
4f404b2 to
bc9fcbd
Compare
|
GNU testsuite comparison: |
0aca569 to
39d9dad
Compare
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
|
|
GNU testsuite comparison: |
f5d29b8 to
ed07374
Compare
|
GNU testsuite comparison: |
044bb60 to
de41913
Compare
|
GNU testsuite comparison: |
d5e6a80 to
944ea9a
Compare
|
GNU testsuite comparison: |
|
@cakebaker @RenjiSann I made a symlink plan. I think we can make individual bins after hashsum deletion. Failing test is |
|
GNU testsuite comparison: |
|
This looks very dangerous option if we don't have an extension |
Alt #9776 . but without individual bins.