Skip to content

Conversation

@trypsynth
Copy link

No description provided.

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/shuf/shuf-reservoir (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/sort/sort-stale-thread-mem (passes in this run but fails in the 'main' branch)

@cakebaker
Copy link
Contributor

Can you please add a test to ensure we don't regress in the future? Thanks.

@trypsynth
Copy link
Author

@cakebaker all done

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 15, 2026

Merging this PR will degrade performance by 6.21%

⚡ 2 improved benchmarks
❌ 2 regressed benchmarks
✅ 278 untouched benchmarks
⏩ 38 skipped benchmarks1

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

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Memory du_deep_tree[(100, 3)] 155.8 KB 164.4 KB -5.23%
Memory du_wide_tree[(5000, 500)] 1.2 MB 1.3 MB -6.21%
Memory numfmt_large_numbers_si[10000] 4.9 MB 4.7 MB +4.01%
Memory dd_copy_partial 133.5 KB 129.1 KB +3.4%

Comparing trypsynth:tail-zero (1894e43) with main (5fd3459)

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.


use crate::forwards_thru_file;
use std::io::Cursor;
use crate::{backwards_thru_file, forwards_thru_file};
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this be an integration test in test_tail.rs? I think it would be more ideal to make sure the behaviour with that specific command doesn't regress, also would have less boilerplate to set up the test conditions

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.

3 participants