> On Aug. 21, 2018, 4:54 p.m., James Peach wrote: > > docs/configuration/agent.md > > Line 1016 (original), 1016 (patched) > > <https://reviews.apache.org/r/68443/diff/2/?file=2075695#file2075695line1016> > > > > How are people reading the docs expected to find `flags.proto`?
Yeah, we should follow up to clean this up. - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68443/#review207671 ----------------------------------------------------------- On Aug. 21, 2018, 4:45 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68443/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2018, 4:45 p.m.) > > > Review request for mesos, Gilbert Song, James Peach, Stéphane Cottin, and > Zhitao Li. > > > Repository: mesos > > > Description > ------- > > Since the unit is nanoseconds, setting it to 3600 is way too small, and > misleading. This patch fixed the example to use 1 hour instead. > > > Diffs > ----- > > docs/configuration/agent.md bcddea2f8606f71fc46a39debca323d85c1d94c1 > src/slave/flags.cpp fd208f64416ae865b81c8ba5950b20bde6be00a3 > > > Diff: https://reviews.apache.org/r/68443/diff/2/ > > > Testing > ------- > > trivial > > > Thanks, > > Jie Yu > >
