-----------------------------------------------------------
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

Reply via email to