-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35335/
-----------------------------------------------------------
(Updated June 18, 2015, 11:11 p.m.)
Review request for mesos, Benjamin Hindman and Joris Van Remoortere.
Changes
-------
Minor syntactical changes
Bugs: MESOS-2598
https://issues.apache.org/jira/browse/MESOS-2598
Repository: mesos
Description
-------
Fixed JSON output error TaskInfo executor_id in slave/http.cpp
Diffs (updated)
-----
src/slave/http.cpp bc25bdd33277dbfa30410ad081ea09f0fc39c598
src/tests/slave_tests.cpp 8709874411835c99b8900102fd014a9e28f2b208
Diff: https://reviews.apache.org/r/35335/diff/
Testing (updated)
-------
Extended SlaveEndpoint test to launch a task and check for correct executor_id
in state.json.
It was difficult to mock queued_tasks for testing (see
https://issues.apache.org/jira/browse/MESOS-2896), so the extension to
SlaveEndpoint verifies the executor_id format for general tasks.
Thanks,
Brendan Chang
