mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mesos ReviewBot <revi...@mesos.apache.org>
Subject Re: Review Request 45000: MESOS-3902: Fix in location header during redirect from non-leader.
Date Sun, 20 Mar 2016 23:21:41 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45000/#review124470
-----------------------------------------------------------



Patch looks great!

Reviews applied: [44985, 45000]

Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1
MESOS_VERBOSE=1'; ./support/docker_build.sh

- Mesos ReviewBot


On March 20, 2016, 11:03 p.m., Ashwin Murthy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45000/
> -----------------------------------------------------------
> 
> (Updated March 20, 2016, 11:03 p.m.)
> 
> 
> Review request for mesos and Vinod Kone.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> MESOS-3902: [Updated] Fix location header in redirect from non-leader. Addressed comments
from Vinod and Ben. Removed the scheme but added the path from the original request to the
location header. Preserved the link to the RFC in the comments.
> 
> 
> Diffs
> -----
> 
>   src/master/http.cpp bfea8fa811c24d37b2d7a8109728e1f40217e02f 
> 
> Diff: https://reviews.apache.org/r/45000/diff/
> 
> 
> Testing
> -------
> 
> Was trying to write unit test in scheduler http api test but it turns out multi master
tests cannot be written at this point. Need to manually verify this by setting up multiple
masters with ZK.
> 
> 
> Thanks,
> 
> Ashwin Murthy
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message