----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43450/#review118776 -----------------------------------------------------------
Patch looks great! Reviews applied: [43450] Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot 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 > >
