----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68321/#review207956 -----------------------------------------------------------
Fix it, then Ship it! Looks good to me (modulo testing). src/master/http.cpp Line 3476 (original), 3487 (patched) <https://reviews.apache.org/r/68321/#comment291623> `return std::move(OK(...));` - Alexander Rukletsov On Aug. 21, 2018, 2:09 p.m., Benno Evers wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68321/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2018, 2:09 p.m.) > > > Review request for mesos, Alexander Rukletsov and Benjamin Mahler. > > > Bugs: MESOS-9158 > https://issues.apache.org/jira/browse/MESOS-9158 > > > Repository: mesos > > > Description > ------- > > Added '/state-summary' to the set of batched master endpoints. > > > Diffs > ----- > > src/master/http.cpp ae28d525e5c0fe067ae77a4c71e8f18f772ff1dc > src/master/master.hpp dc0080b24f19b77a4de34ab24aece657726343b8 > > > Diff: https://reviews.apache.org/r/68321/diff/3/ > > > Testing > ------- > > > Thanks, > > Benno Evers > >
