----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18155/#review34786 -----------------------------------------------------------
Ship it! Looks great! docs/high-availability.md <https://reviews.apache.org/r/18155/#comment65089> "Provide the znode path when starting the masters, slaves, and framework schedulers as follows:" The "to all slaves / scheduler drivers" part is incomplete (ignores masters) and unnecessary. docs/high-availability.md <https://reviews.apache.org/r/18155/#comment65094> Why the newline between [Scheduler and API]? docs/high-availability.md <https://reviews.apache.org/r/18155/#comment65102> I would put "Note that...vice versa." in (parentheses) so that "the following happens" is the last real statement in the paragraph before the following list. - Adam B On Feb. 14, 2014, 3:50 p.m., Tom Galloway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18155/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2014, 3:50 p.m.) > > > Review request for mesos, Dave Lester and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > Updated version of high availability doc. > > > Diffs > ----- > > docs/high-availability.md 77af5d5 > > Diff: https://reviews.apache.org/r/18155/diff/ > > > Testing > ------- > > > Thanks, > > Tom Galloway > >
