Skip to content

[25.1] Discard rest of line in chunks in iter_start_of_line#22332

Merged
mvdbeek merged 2 commits intogalaxyproject:release_25.1from
mvdbeek:fix_readline_to_advance_iterator_25.1
Apr 1, 2026
Merged

[25.1] Discard rest of line in chunks in iter_start_of_line#22332
mvdbeek merged 2 commits intogalaxyproject:release_25.1from
mvdbeek:fix_readline_to_advance_iterator_25.1

Conversation

@mvdbeek
Copy link
Copy Markdown
Member

@mvdbeek mvdbeek commented Mar 31, 2026

Avoid reading the entire remainder of a line into memory when discarding, as some lines can be gigabytes in size.
Fixes #22331

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

Avoid reading the entire remainder of a line into memory when
discarding, as some lines can be gigabytes in size.
@mvdbeek mvdbeek force-pushed the fix_readline_to_advance_iterator_25.1 branch from f24382c to 3358f8f Compare March 31, 2026 19:49
@mvdbeek mvdbeek merged commit 22c1e0e into galaxyproject:release_25.1 Apr 1, 2026
49 of 52 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in Galaxy Dev - weeklies Apr 1, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 1, 2026

This PR was merged without a "kind/" label, please correct.

@nsoranzo nsoranzo deleted the fix_readline_to_advance_iterator_25.1 branch April 1, 2026 08:41
@ahmedhamidawan ahmedhamidawan modified the milestones: 26.1, 26.0 Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

4 participants