> On Oct. 19, 2012, 6:46 p.m., Vinod Kone wrote: > > src/slave/paths.hpp, line 211 > > <https://reviews.apache.org/r/7658/diff/2/?file=177955#file177955line211> > > > > how do you like 'current' instead of 'latest'? i think that's more > > idiomatic?
Discussed with benh and we decided on latest since "latest" makes sense regardless of whether the "latest" executor is running. Whereas "current" is a bit confusing when there's no executor running. > On Oct. 19, 2012, 6:46 p.m., Vinod Kone wrote: > > src/slave/paths.hpp, line 217 > > <https://reviews.apache.org/r/7658/diff/2/?file=177955#file177955line217> > > > > s/mkdir/symlink/ nice catch! - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7658/#review12616 ----------------------------------------------------------- On Oct. 19, 2012, 12:46 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7658/ > ----------------------------------------------------------- > > (Updated Oct. 19, 2012, 12:46 a.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > This replaces our run directory numbers with the executor uuids. > > Ex: > .../executorFoo/runs/0 > .../executorFoo/runs/1111-1111-1111-1111 > > This ensures we don't ever write to an old directory that was gc'ed. > > > Diffs > ----- > > src/slave/paths.hpp 48a7be0ae33785206ebb1985647178224d6bcfb8 > src/slave/slave.hpp 343c353027dbd2a7c1be8cee99a1d59367169177 > src/slave/slave.cpp 5af7464aae17c00a0e707421982d7cb055aabc6c > src/slave/state.hpp dcadd511063584cde51f533e2120f4eab5145fd2 > src/slave/state.cpp c8406fbb5586bca5b7f169a2ef937c958626dee1 > src/tests/slave_state_tests.cpp 0e232ff3ed773f3bdccc05e716b66d106d80fb2f > third_party/libprocess/include/stout/fs.hpp > 1516d0b6fa6e0ced26bb08ab6a4fadf28232124f > > Diff: https://reviews.apache.org/r/7658/diff/ > > > Testing > ------- > > Updated tests + make check. > > > Thanks, > > Ben Mahler > >
