----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70450/#review214558 -----------------------------------------------------------
src/slave/http.cpp Lines 793 (patched) <https://reviews.apache.org/r/70450/#comment300731> Can you add a TODO to actually send the correct framework and executor id in HEARTBEAT calls. It seems weird/wrong that we send dummy values. - Vinod Kone On April 10, 2019, 11:55 p.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70450/ > ----------------------------------------------------------- > > (Updated April 10, 2019, 11:55 p.m.) > > > Review request for mesos, Gastón Kleiman, Greg Mann, and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > This silences a '400 Bad Request' response by the agent whenever an > executor sends a HEARTBEAT call. These HEARTBEATs do not include a > valid value for required fields (FrameworkID and ExecutorID) because > they are not known by the library generating the HEARTBEATs. > > The error is harmless because HEARTBEAT calls do not have any effect > besides generating traffic. > > > Diffs > ----- > > src/slave/http.cpp d66ae520f68b783caea8937a1a753608099d5926 > > > Diff: https://reviews.apache.org/r/70450/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Joseph Wu > >
