Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/#review97909 --- src/master/http.cpp (lines 1909 - 1914)

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/#review97894 --- src/master/http.cpp (lines 1909 - 1914)

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/ --- (Updated Sept. 4, 2015, 11:20 p.m.) Review request for mesos, Adam B, Benjamin

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/ --- (Updated Aug. 12, 2015, 2:46 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/#review90790 --- Ship it! LGTM! - Jie Yu On Aug. 5, 2015, 7:12 p.m., Michael Park

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/ --- (Updated Aug. 5, 2015, 7:12 p.m.) Review request for mesos, Adam B, Benjamin Hi

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/ --- (Updated Aug. 5, 2015, 11:15 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/ --- (Updated Aug. 5, 2015, 10:44 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/ --- (Updated Aug. 5, 2015, 9:54 a.m.) Review request for mesos, Adam B, Benjamin Hi

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/ --- (Updated July 31, 2015, 9:57 p.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

> On July 16, 2015, 3:04 p.m., Alexander Rukletsov wrote: > > src/master/http.cpp, lines 1325-1332 > > > > > > Why do we need to recover resources for unreserve? > > Michael Park wrote: > If reserved resources ar

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

> On July 16, 2015, 3:04 p.m., Alexander Rukletsov wrote: > > src/master/http.cpp, lines 1325-1332 > > > > > > Why do we need to recover resources for unreserve? > > Michael Park wrote: > If reserved resources ar

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/ --- (Updated July 28, 2015, 9:06 p.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

> On July 2, 2015, 3:39 p.m., Alexander Rukletsov wrote: > > A high level question: do operators have the possibility to get a list of > > all dynamic reservations? I think about a situation, when a framework made > > some reservations and then quit, an operator wants to clean up those > > res

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

> On July 16, 2015, 3:04 p.m., Alexander Rukletsov wrote: > > src/master/http.cpp, lines 1325-1332 > > > > > > Why do we need to recover resources for unreserve? > > Michael Park wrote: > If reserved resources ar

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

> On July 16, 2015, 3:04 p.m., Alexander Rukletsov wrote: > > src/master/http.cpp, line 1291 > > > > > > As in https://reviews.apache.org/r/35702/, I suggest we extract > > validation into a separate function. > > M

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

> On July 16, 2015, 3:04 p.m., Alexander Rukletsov wrote: > > src/master/http.cpp, lines 1325-1332 > > > > > > Why do we need to recover resources for unreserve? If reserved resources are offered, we need to recover

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

> On July 2, 2015, 3:39 p.m., Alexander Rukletsov wrote: > > A high level question: do operators have the possibility to get a list of > > all dynamic reservations? I think about a situation, when a framework made > > some reservations and then quit, an operator wants to clean up those > > res

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/#review91890 --- src/master/http.cpp (line 1291)

Re: Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/#review90235 --- A high level question: do operators have the possibility to get a li

Review Request 35983: Added /unreserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35983/ --- Review request for mesos, Adam B, Benjamin Hindman, Ben Mahler, Jie Yu, Joris Va