mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benno Evers <bev...@mesosphere.com>
Subject Re: Review Request 69071: Narrowed interface of `ReadOnlyHandler` members.
Date Thu, 22 Nov 2018 01:19:23 GMT

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

(Updated Nov. 22, 2018, 1:19 a.m.)


Review request for mesos, Alexander Rukletsov and Joseph Wu.


Changes
-------

Rebased onto latest master.


Repository: mesos


Description (updated)
-------

Previously, the members of `ReadOnlyHandler` would take a full `Request` as parameter, making
it hard for clients to reason about which parts of the request are used internally, and even
harder to guarantee that behaviour into the future.

This commit changes the interface so only the query parameters get passed.


Diffs (updated)
-----

  src/master/http.cpp 68ee2a6dcffbc772afec6e797b1af8da48f61937 
  src/master/master.hpp 3b3c1a4e61de9503c8d038dd3bee623ded5914c9 
  src/master/readonly_handler.cpp 8895374499dc6baa2c4d8a8dd86fddac4e39be29 


Diff: https://reviews.apache.org/r/69071/diff/2/

Changes: https://reviews.apache.org/r/69071/diff/1-2/


Testing
-------


Thanks,

Benno Evers


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