----------------------------------------------------------- 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. Changes ------- Tweak. Repository: mesos Description (updated) ------- 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 (updated) ----- 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