----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63917/#review191421 -----------------------------------------------------------
Fix it, then Ship it! src/resource_provider/message.hpp Line 75 (original), 75 (patched) <https://reviews.apache.org/r/63917/#comment269237> Sounds like the extra `<<` in the middle is not needed. - Jie Yu On Nov. 17, 2017, 5:24 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63917/ > ----------------------------------------------------------- > > (Updated Nov. 17, 2017, 5:24 p.m.) > > > Review request for mesos, Chun-Hung Hsiao, Jie Yu, and Jan Schlicht. > > > Repository: mesos > > > Description > ------- > > This message now captures much more than just resource updates, but > instead informs users about the full resource provider state. This > change mirrors the message naming used between resource provider > manager and resource providers. > > > Diffs > ----- > > src/resource_provider/manager.cpp 6dfc42900a1e4249f37cec585f7fe50f5aa94e43 > src/resource_provider/message.hpp 05879cdc01dca88440ce89beb487168c033fab37 > src/slave/slave.cpp d8edc5e6bbfa265bca4d19bbaa7db3063949dbc0 > src/tests/resource_provider_manager_tests.cpp > ecfe2b4c0952838d6312df603f8eb2f458725175 > > > Diff: https://reviews.apache.org/r/63917/diff/1/ > > > Testing > ------- > > > Thanks, > > Benjamin Bannier > >
