----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68945/#review209302 -----------------------------------------------------------
PASS: Mesos patch 68945 was successfully built and tested. Reviews applied: `['68944', '68945']` All the build artifacts available at: http://dcos-win.westus2.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2433/mesos-review-68945 - Mesos Reviewbot Windows On Oct. 8, 2018, 9:50 a.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68945/ > ----------------------------------------------------------- > > (Updated Oct. 8, 2018, 9:50 a.m.) > > > Review request for mesos and Benno Evers. > > > Repository: mesos > > > Description > ------- > > There is no good reason to provide two c-tors for `MethodNotAllowed`, > with one taking `requestMethod` and one not. Instead, an `Option<>` > can be used. This also removes the need for copy-paste in the c-tor > body. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/http.hpp > cef511a1563a26e170ce7f4a49de12776b4512e7 > > > Diff: https://reviews.apache.org/r/68945/diff/2/ > > > Testing > ------- > > `make check` on various linux distros > > > Thanks, > > Alexander Rukletsov > >