Re: Review Request 40906: Replaced `BadRequest` with `MethodNotAllowed` for all HTTP requests with unsupported methods.

2015-12-10 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40906/#review109757 --- Patch looks great! Reviews applied: [40905, 40913, 40906] Passed

Re: Review Request 40906: Replaced `BadRequest` with `MethodNotAllowed` for all HTTP requests with unsupported methods.

2015-12-09 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40906/ --- (Updated Dec. 10, 2015, 12:13 a.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 40906: Replaced `BadRequest` with `MethodNotAllowed` for all HTTP requests with unsupported methods.

2015-12-09 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40906/#review109693 --- Ship it! Ship It! - Anand Mazumdar On Dec. 10, 2015, 12:13

Review Request 40906: Replaced `BadRequest` with `MethodNotAllowed` for all HTTP requests with unsupported methods.

2015-12-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40906/ --- Review request for mesos, Anand Mazumdar, Isabel Jimenez, and Joris Van

Re: Review Request 40906: Replaced `BadRequest` with `MethodNotAllowed` for all HTTP requests with unsupported methods.

2015-12-03 Thread Guangya Liu
> On 十二月 3, 2015, 12:22 p.m., Guangya Liu wrote: > > src/master/http.cpp, line 555 > > > > > > I see that others using "Expecting a 'POST' request", this may be more > > clear > > > > Also, is it a MUST to

Re: Review Request 40906: Replaced `BadRequest` with `MethodNotAllowed` for all HTTP requests with unsupported methods.

2015-12-03 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40906/#review108811 --- src/master/http.cpp (line 555)

Re: Review Request 40906: Replaced `BadRequest` with `MethodNotAllowed` for all HTTP requests with unsupported methods.

2015-12-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40906/#review108826 --- Bad patch! Reviews applied: [40905] Failed command:

Re: Review Request 40906: Replaced `BadRequest` with `MethodNotAllowed` for all HTTP requests with unsupported methods.

2015-12-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40906/ --- (Updated Dec. 3, 2015, 2:35 p.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 40906: Replaced `BadRequest` with `MethodNotAllowed` for all HTTP requests with unsupported methods.

2015-12-03 Thread Alexander Rukletsov
> On Dec. 3, 2015, 12:22 p.m., Guangya Liu wrote: > > src/master/http.cpp, line 555 > > > > > > I see that others using "Expecting a 'POST' request", this may be more > > clear > > > > Also, is it a MUST