Make chunk sizes configurable for ProcessBuilder
Nobody has claimed this yet.
- Dominant language
- Scala
- Stars
- 2.5k
- Forks
- 636
- Avg merge
- 2d 4h
- Merged PRs (30d)
- 7
Description
The JVM implementation for the stdout and stderr streams for Processes generated by ProcessBuilder.spawn currently use a chunk size of 8192. While this is probably fine for most use cases I've noticed that it can limit throughput drastically when large amounts of data are transferred this way.
It would be helpful, if the chunk size would be a parameter, either of ProcessBuilder or by having a variant of stderr that takes a chunkSize parameter like
def stdout(chunkSize: Int = 8192): Stream[F, Byte]
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the JVM implementation of ProcessBuilder.spawn and inspect how stdout and stderr currently read 8192-byte chunks. Check how the existing stdout and stderr APIs are tested, then make the chunk size configurable while preserving the current default of 8192 and verify large transfers use the requested size.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- scala
- Domain
- operating-systems
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100