Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-03-24 Thread Michael Park
> On March 10, 2016, 7 p.m., Michael Park wrote: > > src/slave/http.cpp, line 105 > > > > > > If we were using `model` before, we need to maintain that. > > > > We have a `json` defined for `CommandInfo` in

Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-03-24 Thread Michael Park
> On March 24, 2016, 4:38 p.m., Michael Park wrote: > > src/slave/http.cpp, line 96 > > > > > > This should be marked `static`, right? This is addressed in https://reviews.apache.org/r/44981/. - Michael

Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-03-19 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43911/ --- (Updated March 17, 2016, 9:33 p.m.) Review request for mesos and Michael Park.

Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-03-18 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43911/ --- (Updated March 16, 2016, 9:14 p.m.) Review request for mesos and Michael Park.

Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-03-11 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43911/ --- (Updated March 11, 2016, 10:33 p.m.) Review request for mesos and Michael

Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-03-10 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43911/#review122958 --- src/slave/http.cpp (lines 89 - 105)

Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-02-28 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43911/ --- (Updated Feb. 29, 2016, 7:01 a.m.) Review request for mesos and Michael Park.

Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-02-28 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43911/ --- (Updated Feb. 29, 2016, 6:55 a.m.) Review request for mesos and Michael Park.

Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-02-27 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43911/ --- (Updated Feb. 28, 2016, 12:16 a.m.) Review request for mesos and Michael Park.

Re: Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-02-23 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43911/#review120453 --- Patch looks great! Reviews applied: [43816, 43822, 43823, 43817,

Review Request 43911: Updated `/state` agent endpoint to use jsonify.

2016-02-23 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43911/ --- Review request for mesos and Michael Park. Repository: mesos Description