----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69472/#review211076 -----------------------------------------------------------
Fix it, then Ship it! Looks great, thanks Joseph! src/slave/http.cpp Line 834 (original), 834-835 (patched) <https://reviews.apache.org/r/69472/#comment295948> Nit: not indented far enough after open paren. - Greg Mann On Dec. 6, 2018, 1:23 a.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69472/ > ----------------------------------------------------------- > > (Updated Dec. 6, 2018, 1:23 a.m.) > > > Review request for mesos, Benno Evers, Gastón Kleiman, and Greg Mann. > > > Bugs: MESOS-7564 > https://issues.apache.org/jira/browse/MESOS-7564 > > > Repository: mesos > > > Description > ------- > > This moves the very similar `HttpConnection` classes inside the > master and agent into a common header. The refactored > `StreamingHttpConnection<Event>` is more explicitly named to avoid > potentially clashing with the libprocess HTTP helpers. > > This also moves the master's heartbeater helper into a new header > and transforms it into an RAII libprocess actor wrapper. The > heartbeater depends on this `StreamingHttpConnection` and is currently > used by the master for heartbeating the operator event stream > and HTTP framework connection. A later patch will use this heartbeater > for agent->executor heartbeats. > > > Diffs > ----- > > src/common/heartbeater.hpp PRE-CREATION > src/common/http.hpp ac9ed5e8b1c6e8e100eba401136eb95c9dd621d4 > src/master/framework.cpp 36eda9f287bef28608024e0036db1f955a1b393c > src/master/http.cpp 68ee2a6dcffbc772afec6e797b1af8da48f61937 > src/master/master.hpp c7becfa615964674dcf1ebd9424aa5818a0fdb85 > src/master/master.cpp 3b3824a67f46866cd64e32d7f9f92484b5891aa2 > src/slave/http.cpp ba389263ef7ab311682d542424eb56360841d24c > src/slave/slave.hpp edf7269d4057ec8c95bb54c855210ad00d002a50 > src/slave/slave.cpp 858b78620e1ef33f3587d0bd95a684917aaf5bbb > > > Diff: https://reviews.apache.org/r/69472/diff/3/ > > > Testing > ------- > > make > > The original plan was to use the same helper for the executor too, but of the > Master, Framework, and Agent heartbeats, only the executor lacks a streaming > connection. > > > Thanks, > > Joseph Wu > >