Re: Review Request 64639: Supported getting all containers in the agent API.

2017-12-15 Thread Jie Yu
> On Dec. 15, 2017, 9:46 p.m., Greg Mann wrote: > > src/slave/http.cpp > > Lines 2275-2276 (original), 2380-2384 (patched) > > > > > > Nit: these fit on one line. The second one does not :( thus the change. -

Re: Review Request 64639: Supported getting all containers in the agent API.

2017-12-15 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64639/#review193965 --- Fix it, then Ship it! src/slave/http.cpp Lines 2327 (patched)

Re: Review Request 64639: Supported getting all containers in the agent API.

2017-12-15 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64639/#review193942 --- FAIL: Some Mesos tests failed. Reviews applied: `['64623',

Re: Review Request 64639: Supported getting all containers in the agent API.

2017-12-15 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64639/ --- (Updated Dec. 15, 2017, 4:56 p.m.) Review request for mesos, Gilbert Song,

Re: Review Request 64639: Supported getting all containers in the agent API.

2017-12-14 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64639/#review193882 --- FAIL: Some Mesos tests failed. Reviews applied: `['64623',

Review Request 64639: Supported getting all containers in the agent API.

2017-12-14 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64639/ --- Review request for mesos, Gilbert Song, Joseph Wu, and Vinod Kone. Repository: