mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ashwin Murthy <ashwinmur...@gmail.com>
Subject Re: Review Request 45000: MESOS-3902: Fix in location header during redirect from non-leader.
Date Sun, 20 Mar 2016 23:03:44 GMT

-----------------------------------------------------------
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 (updated)
-------

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 (updated)
-----

  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