----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46990/#review131783 -----------------------------------------------------------
Bad patch! Reviews applied: [46990, 46989, 46882, 46881, 46880, 46876, 46870, 46869, 46883, 46868, 46867, 46866] Failed command: ./support/apply-review.sh -n -r 46883 Error: 2016-05-05 04:58:11 URL:https://reviews.apache.org/r/46883/diff/raw/ [1105/1105] -> "46883.patch" [1] error: patch failed: src/common/http.cpp:589 error: src/common/http.cpp: patch does not apply Full log: https://builds.apache.org/job/mesos-reviewbot/12891/console - Mesos ReviewBot On May 4, 2016, 6:48 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46990/ > ----------------------------------------------------------- > > (Updated May 4, 2016, 6:48 p.m.) > > > Review request for mesos, Alexander Rojas and Kapil Arya. > > > Bugs: MESOS-5286 > https://issues.apache.org/jira/browse/MESOS-5286 > > > Repository: mesos > > > Description > ------- > > The previous commit added authorization handling > to `ProcessBase::visit`, so the authorization > handlers in the endpoint code can be removed. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/logging.hpp > 651cb3736705124bec9e4366c0e7adc5d5b488f0 > 3rdparty/libprocess/include/process/metrics/metrics.hpp > 25e8fcaeb9c71eaa43096165745e79c8198c139a > 3rdparty/libprocess/src/logging.cpp > ad7aa3aabbfc4a496e7e624a691a72e45c72e35d > 3rdparty/libprocess/src/metrics/metrics.cpp > 65b58433158c51cebace9b591d4121e8d9a59ecc > > Diff: https://reviews.apache.org/r/46990/diff/ > > > Testing > ------- > > `make check` on OSX. > > > Thanks, > > Greg Mann > >