Skip to content

Prevent LimitOverrunError with large output lines

Placeholder Chris Harris requested to merge github/fork/cjh1/limit-overrun into limit_over_run_outpu

@ mention of reviewers`

@ihsaan-ullah`

A brief description of the purpose of the changes contained in this PR.

If a submission writes a output line larger than the stream buffer size ( default 64k ) a LimitOverrunError will be raise. Rather than using readline(...) use readutil(....) and in the case of a overrun just return the current buffer, the rest of the line will be returned with the next read.

A checklist for hand testing

  • Tested with local worker connected to codabench.org

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

Merge request reports