----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43450/#review120052 -----------------------------------------------------------
Ship it! Ship It! - Ben Mahler On Feb. 10, 2016, 11:16 p.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43450/ > ----------------------------------------------------------- > > (Updated Feb. 10, 2016, 11:16 p.m.) > > > Review request for mesos and Ben Mahler. > > > Repository: mesos > > > Description > ------- > > When the `uuid` field of `StatusUpdate` is not set, the output of > `operator<<(ostream& stream, const StatusUpdate& update)` contains a > mismatched parenthesis. In practice, this means we can omit log > messages that have a typo. > > > Diffs > ----- > > src/messages/messages.cpp 89deee34948bcf0fadcabc0fbdd4356c0bd2ddb7 > > Diff: https://reviews.apache.org/r/43450/diff/ > > > Testing > ------- > > make check > > Old log output: > > ``` > Sending status update TASK_LOST) for task > 33fa8ceb-f133-43c0-8a8d-9a77f83b825b of framework > 06daa081-8522-42b1-a4d3-1160c1f0662d-0000 'Task launched with invalid offers: > Offer 06daa081-8522-42b1-a4d3-1160c1f0662d-O1 is no longer valid' > ``` > > New log output: > > ``` > Sending status update TASK_LOST for task f3674b8a-5242-4864-8bab-cbb5f342de05 > of framework 5bc9cf4c-4767-4800-92e1-b6b56d17edd3-0000 'Task launched with > invalid offers: Offer 5bc9cf4c-4767-4800-92e1-b6b56d17edd3-O1 is no longer > valid' > ``` > > > Thanks, > > Neil Conway > >