This is an automatically generated e-mail. To reply, visit:

(Updated May 17, 2016, 8:34 p.m.)

Review request for mesos, Benjamin Mahler, Joris Van Remoortere, and switched 
to 'mcypark'.


Changed the logic in the read() call to copy from a stack allocated buffer 
instead of writing to the string directly. I usually don't like stack allocated 
buffers because of their potential security implications when it comes to stack 
buffer overflows, but I took extra care here to make sure there won't be any :).

Bugs: MESOS-5398

Repository: mesos


The previous read() implementation was based on calling getline() to
read in chunks of data from a file. This is fine for text-based files,
but is a little strange for binary files.

The new implementation reads in chunks of raw bytes in exponentially
increasing chunk sizes, and works well for both text files and binary
files alike.

Diffs (updated)


Diff: https://reviews.apache.org/r/47481/diff/

Testing (updated)

make check -j


Kevin Klues

Reply via email to