----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37813/#review102435 -----------------------------------------------------------
Ship it! src/tests/fetcher_cache_tests.cpp (line 567) <https://reviews.apache.org/r/37813/#comment160088> To prevent further confusion, wouldn't it make sense to call this `sandboxPath` here and everywhere else instead - like you did before? - Till Toenshoff On Oct. 12, 2015, 4:25 p.m., Bernd Mathiske wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37813/ > ----------------------------------------------------------- > > (Updated Oct. 12, 2015, 4:25 p.m.) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, Joseph Wu, and Till > Toenshoff. > > > Bugs: MESOS-3235 > https://issues.apache.org/jira/browse/MESOS-3235 > > > Repository: mesos > > > Description > ------- > > Dumps all involved task/executor sandbox contents in test tear down > only if a failure occurred. > > > Diffs > ----- > > src/tests/fetcher_cache_tests.cpp 7e194dc6e2b2d8b857e61d1a18d696545a86ce9f > > Diff: https://reviews.apache.org/r/37813/diff/ > > > Testing > ------- > > make check on OSX, where the bug showed up. > > > Thanks, > > Bernd Mathiske > >
