mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jacob Janco <>
Subject Re: Review Request 46504: Constructed error string in MethodNotAllowed.
Date Tue, 03 May 2016 00:20:10 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated May 3, 2016, 12:20 a.m.)

Review request for mesos and Alexander Rukletsov.


Addressed issues, nested MethodNotAllowedBody inside MethodNotAllowed.

Bugs: MESOS-4126

Repository: mesos

Description (updated)

Constructed error string in MethodNotAllowed.

- Decided to use constructor delegation rather than just constructing in MethodNotAllowed
  allowing body to be constructed before calling the Response constructor. This in addition
  to the improved readability and sectioning off the verbose body construction as a middle
  in a delegation chain led me to this implementation. Noting here that construction in 
  MethodNotAllowed would be completely valid, but seeing what others think of this choice.

Diffs (updated)

  3rdparty/libprocess/include/process/http.hpp 8f4eabcbb71ead1f5c28e1d8a2dd40db7af1f297 


Testing (updated)

make check


Jacob Janco

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