Re: Review Request 37584: Fix bug accessing error() when no Error

2015-08-18 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37584/#review95754 --- src/launcher/fetcher.cpp (lines 100 - 106)

Re: Review Request 37584: Fix bug accessing error() when no Error

2015-08-18 Thread Marco Massenzio
On Aug. 18, 2015, 6:34 p.m., Vinod Kone wrote: src/launcher/fetcher.cpp, lines 100-106 https://reviews.apache.org/r/37584/diff/1/?file=1043203#file1043203line100 just do return Error(Skipping fetch with Hadoop client: + (available.isError() ?

Re: Review Request 37584: Fix bug accessing error() when no Error

2015-08-18 Thread Ben Mahler
On Aug. 18, 2015, 6:34 p.m., Vinod Kone wrote: src/launcher/fetcher.cpp, lines 100-106 https://reviews.apache.org/r/37584/diff/1/?file=1043203#file1043203line100 just do return Error(Skipping fetch with Hadoop client: + (available.isError() ?

Re: Review Request 37584: Fix bug accessing error() when no Error

2015-08-18 Thread Vinod Kone
On Aug. 18, 2015, 6:34 p.m., Vinod Kone wrote: src/launcher/fetcher.cpp, lines 100-106 https://reviews.apache.org/r/37584/diff/1/?file=1043203#file1043203line100 just do return Error(Skipping fetch with Hadoop client: + (available.isError() ?