----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65025/#review195103 -----------------------------------------------------------
Ship it! Could you explain in the description why we did this? - Benjamin Mahler On Jan. 8, 2018, 6:14 p.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65025/ > ----------------------------------------------------------- > > (Updated Jan. 8, 2018, 6:14 p.m.) > > > Review request for mesos and Benjamin Mahler. > > > Bugs: MESOS-8375 > https://issues.apache.org/jira/browse/MESOS-8375 > > > Repository: mesos > > > Description > ------- > > Replaced `os::read` with `state::read`. > > > Diffs > ----- > > src/slave/containerizer/mesos/isolators/docker/volume/isolator.cpp > deacffeef256bc05aae5325fdc19a11af5d508ba > src/slave/containerizer/mesos/paths.cpp > d6ea618b20431ac95f880045143d09366f1740bf > src/slave/state.cpp 5428b341b061c8209a5cfe63e17b71467d9f4c48 > src/tests/slave_recovery_tests.cpp e305d7430b64409fdac84a33ee072707471d2c49 > > > Diff: https://reviews.apache.org/r/65025/diff/1/ > > > Testing > ------- > > > Thanks, > > Michael Park > >