Re: Review Request 46725: Set default Content-Type for HTTP responses.

2016-05-10 Thread Vinod Kone
> On April 27, 2016, 11:44 p.m., Anand Mazumdar wrote: > > LGTM, just 2 minor nits. > > > > Also, as per our offline discussion: This doesn't handle scenarios where a > > client explicitly expects a error response encoded in a particular type > > specified via the `Accept` header. We would

Re: Review Request 46725: Set default Content-Type for HTTP responses.

2016-04-27 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46725/#review130867 --- Fix it, then Ship it! LGTM, just 2 minor nits. Also, as per

Re: Review Request 46725: Set default Content-Type for HTTP responses.

2016-04-27 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46725/#review130732 --- Patch looks great! Reviews applied: [46725] Passed command:

Review Request 46725: Set default Content-Type for HTTP responses.

2016-04-26 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46725/ --- Review request for mesos and Anand Mazumdar. Bugs: MESOS-3739