mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bernd Mathiske" <>
Subject Re: Review Request 30609: Added a function that reports file size, not following links.
Date Thu, 30 Apr 2015 20:38:04 GMT

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

(Updated April 30, 2015, 1:38 p.m.)

Review request for mesos, Adam B, Benjamin Hindman, Till Toenshoff, and Timothy Chen.


Added test comment.

Bugs: MESOS-2072

Repository: mesos


This returns a file's size (on UNIXes as reported by lstat(), not stat()). It is desired that
in case of a link, the size of the link, not the size of the referenced file, is returned.

Diffs (updated)

  3rdparty/libprocess/3rdparty/stout/include/stout/os/stat.hpp 270c4c848fc0460dcdb9a90823281d735f4550c2

  3rdparty/libprocess/3rdparty/stout/tests/os_tests.cpp 343f95be7f316170b37c9358627f3c2090f0e29e



Wrote a simple test that creates a file and tests its size, and also checks if a non-existing
file yields an error.


Bernd Mathiske

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message