Re: Review Request 45327: Drop HTTP Status `404` and `307` in the scheduler library.

2016-03-25 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45327/#review125435 --- Ship it! Ship It! - Vinod Kone On March 25, 2016, 2:28

Re: Review Request 45327: Drop HTTP Status `404` and `307` in the scheduler library.

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

Re: Review Request 45327: Drop HTTP Status `404` and `307` in the scheduler library.

2016-03-24 Thread Yong Tang
> On March 25, 2016, 1:53 a.m., Anand Mazumdar wrote: > > LGTM, just some minor code comments. Thanks a lot for the review Anand. I just updated the review request with the fixes. Let me know if there are other issues. - Yong --- This

Re: Review Request 45327: Drop HTTP Status `404` and `307` in the scheduler library.

2016-03-24 Thread Yong Tang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45327/ --- (Updated March 25, 2016, 2:28 a.m.) Review request for mesos, Anand Mazumdar

Re: Review Request 45327: Drop HTTP Status `404` and `307` in the scheduler library.

2016-03-24 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45327/#review125378 --- Fix it, then Ship it! LGTM, just some minor code comments.