----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37584/#review95754 -----------------------------------------------------------
src/launcher/fetcher.cpp (lines 100 - 106) <https://reviews.apache.org/r/37584/#comment150866> just do return Error("Skipping fetch with Hadoop client: " + (available.isError() ? availabe.error() : " client not found")); - Vinod Kone On Aug. 18, 2015, 5:24 p.m., Marco Massenzio wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37584/ > ----------------------------------------------------------- > > (Updated Aug. 18, 2015, 5:24 p.m.) > > > Review request for mesos and Adam B. > > > Bugs: MESOS-3287 > https://issues.apache.org/jira/browse/MESOS-3287 > > > Repository: mesos > > > Description > ------- > > When trying to download an artifact with the Hadoop client, > and the client is not `available()` we correctly return a > false value, but then in the error message, we tried to > make a call to `Try::error` which failed and crashed Master. > > This fixes it. > > > Diffs > ----- > > src/launcher/fetcher.cpp f8b46b8f957942d0cfdc45d3361f52dae3e514a3 > > Diff: https://reviews.apache.org/r/37584/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Marco Massenzio > >
