Re: Review Request 43450: Fixed typo in log message.
--- 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- '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- 'Task launched with > invalid offers: Offer 5bc9cf4c-4767-4800-92e1-b6b56d17edd3-O1 is no longer > valid' > ``` > > > Thanks, > > Neil Conway > >
Re: Review Request 43450: Fixed typo in log message.
--- 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- '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- 'Task launched with > invalid offers: Offer 5bc9cf4c-4767-4800-92e1-b6b56d17edd3-O1 is no longer > valid' > ``` > > > Thanks, > > Neil Conway > >
Re: Review Request 43450: Fixed typo in log message.
--- 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- '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- 'Task launched with invalid offers: Offer 5bc9cf4c-4767-4800-92e1-b6b56d17edd3-O1 is no longer valid' ``` Thanks, Neil Conway
Review Request 43450: Fixed typo in log message.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43450/ --- Review request for mesos and Ben Mahler. Repository: mesos Description --- When the `StatusUpdate` doesn't contain a UUID field, the log message previously contained an unmatched parenthesis. 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- '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- 'Task launched with invalid offers: Offer 5bc9cf4c-4767-4800-92e1-b6b56d17edd3-O1 is no longer valid' ``` Thanks, Neil Conway