26735429103
88%

Ran 01 Jun 2026 04:38AM UTC

Jobs 0

Files 0

Run time –

Badge

Embed ▾

pending completion

Build # 26735429103

Build Type

push

github

Committed by

web-flow

Commit Message

fix: O(n^2) performance in StdinDataRedirection.popen (#798)

* fix: write stdin data in one call instead of O(n^2) chunked loop

StdinDataRedirection.popen wrote data to a TemporaryFile in 16 KB
chunks using `data = data[CHUNK_SIZE:]`. Each slice copies the entire
remaining buffer, making the total work O(n^2). For a 242 MB input,
this means ~1.8 TB of memory copies and takes hours.

Replace the loop with a single `f.write(data)` call.
TemporaryFile.write() handles arbitrarily large data, and the data
is already fully materialized in memory, so chunking served no
purpose.

Fixes #797

* fix: remove used CHUNK_SIZE

Signed-off-by: Henry Schreiner <henryfs@princeton.edu>

---------

Signed-off-by: Henry Schreiner <henryfs@princeton.edu>
Co-authored-by: Henry Schreiner <henryfs@princeton.edu>

Coverage Stats

Source Files on build 26735429103

Detailed source file information is not available for this build.

tomerfiliba / plumbum / 26735429103
88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 26735429103

tomerfiliba / plumbum / 26735429103 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 26735429103

tomerfiliba / plumbum / 26735429103
88%

README BADGES
x