You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Benjamin Hindman <be...@berkeley.edu> on 2013/12/02 18:26:56 UTC
Re: Review Request 15540: Added master/tasks.json endpoint.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15540/#review29594
-----------------------------------------------------------
Ship it!
Ship It!
- Benjamin Hindman
On Nov. 26, 2013, 6 a.m., Niklas Nielsen wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15540/
> -----------------------------------------------------------
>
> (Updated Nov. 26, 2013, 6 a.m.)
>
>
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Ross Allen.
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> This patch lists all running and recent completed tasks, sorted by date
> and starting at offset and limited by limit parameter.
>
>
> Diffs
> -----
>
> src/master/constants.hpp 634067a
> src/master/constants.cpp 4475a0d
> src/master/http.cpp 218906a
> src/master/master.hpp 2d5dfaf
> src/master/master.cpp f13d2a0
>
> Diff: https://reviews.apache.org/r/15540/diff/
>
>
> Testing
> -------
>
> make check and functional testing
>
>
> Thanks,
>
> Niklas Nielsen
>
>
Re: Review Request 15540: Added master/tasks.json endpoint.
Posted by Benjamin Hindman <be...@gmail.com>.
No objections!
> On Dec 4, 2013, at 2:42 PM, "Niklas Nielsen" <ni...@qni.dk> wrote:
>
>
>
>>> On Dec. 2, 2013, 5:26 p.m., Benjamin Hindman wrote:
>>> Ship It!
>
> I'll go ahead and commit this too if you have no further objections :)
>
>
> - Niklas
>
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15540/#review29594
> -----------------------------------------------------------
>
>
>> On Nov. 26, 2013, 6 a.m., Niklas Nielsen wrote:
>>
>> -----------------------------------------------------------
>> This is an automatically generated e-mail. To reply, visit:
>> https://reviews.apache.org/r/15540/
>> -----------------------------------------------------------
>>
>> (Updated Nov. 26, 2013, 6 a.m.)
>>
>>
>> Review request for mesos, Benjamin Hindman, Ben Mahler, and Ross Allen.
>>
>>
>> Repository: mesos-git
>>
>>
>> Description
>> -------
>>
>> This patch lists all running and recent completed tasks, sorted by date
>> and starting at offset and limited by limit parameter.
>>
>>
>> Diffs
>> -----
>>
>> src/master/constants.hpp 634067a
>> src/master/constants.cpp 4475a0d
>> src/master/http.cpp 218906a
>> src/master/master.hpp 2d5dfaf
>> src/master/master.cpp f13d2a0
>>
>> Diff: https://reviews.apache.org/r/15540/diff/
>>
>>
>> Testing
>> -------
>>
>> make check and functional testing
>>
>>
>> Thanks,
>>
>> Niklas Nielsen
>
Re: Review Request 15540: Added master/tasks.json endpoint.
Posted by Niklas Nielsen <ni...@qni.dk>.
> On Dec. 2, 2013, 5:26 p.m., Benjamin Hindman wrote:
> > Ship It!
I'll go ahead and commit this too if you have no further objections :)
- Niklas
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15540/#review29594
-----------------------------------------------------------
On Nov. 26, 2013, 6 a.m., Niklas Nielsen wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15540/
> -----------------------------------------------------------
>
> (Updated Nov. 26, 2013, 6 a.m.)
>
>
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Ross Allen.
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> This patch lists all running and recent completed tasks, sorted by date
> and starting at offset and limited by limit parameter.
>
>
> Diffs
> -----
>
> src/master/constants.hpp 634067a
> src/master/constants.cpp 4475a0d
> src/master/http.cpp 218906a
> src/master/master.hpp 2d5dfaf
> src/master/master.cpp f13d2a0
>
> Diff: https://reviews.apache.org/r/15540/diff/
>
>
> Testing
> -------
>
> make check and functional testing
>
>
> Thanks,
>
> Niklas Nielsen
>
>