----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59428/#review175594 -----------------------------------------------------------
Fix it, then Ship it! 3rdparty/libprocess/src/process.cpp Line 3787 (original), 3787 (patched) <https://reviews.apache.org/r/59428/#comment248972> The user intends to use a functionality which is denied by libprocess, I feel this should be handled with a higher severity. Should this be a warning or even an error instead? 3rdparty/libprocess/src/process.cpp Line 3832 (original), 3832 (patched) <https://reviews.apache.org/r/59428/#comment248973> See above. - Till Toenshoff On May 20, 2017, 4:44 p.m., Anand Mazumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59428/ > ----------------------------------------------------------- > > (Updated May 20, 2017, 4:44 p.m.) > > > Review request for mesos, Greg Mann and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > This ensures ease of debugging when a HTTP request fails due to > access permissions (4xx errors) or when the path is not found. > > > Diffs > ----- > > 3rdparty/libprocess/src/process.cpp > 96ce7dbc486a2f1d55d2238a8a102bf024b12b1c > > > Diff: https://reviews.apache.org/r/59428/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Anand Mazumdar > >
