Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review92052 --- Ship it! Ship It! - Bernd Mathiske On July 17, 2015, 2:05 a.m.,

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review92051 --- Patch looks great! Reviews applied: [36547] All tests passed. - M

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review92048 --- Ship it! Ship It! - Joerg Schad On July 17, 2015, 9:05 a.m., Jan

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/ --- (Updated July 17, 2015, 11:05 a.m.) Review request for mesos, Bernd Mathiske an

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review92046 --- src/launcher/fetcher.cpp (line 126)

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review92045 --- Ship it! Ship It! - Joerg Schad On July 17, 2015, 8:56 a.m., Jan

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/ --- (Updated July 17, 2015, 10:56 a.m.) Review request for mesos, Bernd Mathiske an

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review92042 --- Ship it! Ship It! - Joerg Schad On July 17, 2015, 8:40 a.m., Jan

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/ --- (Updated July 17, 2015, 10:40 a.m.) Review request for mesos, Bernd Mathiske an

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review92041 --- src/launcher/fetcher.cpp (line 120)

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/ --- (Updated July 17, 2015, 10:32 a.m.) Review request for mesos, Bernd Mathiske an

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

> On July 16, 2015, 7:41 p.m., Joerg Schad wrote: > > src/launcher/fetcher.cpp, line 134 > > > > > > I assume those are spaces? Could you please doublecheck? These are spaces. It's probably like this to indicate tha

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

> On July 17, 2015, 4:59 a.m., Adam B wrote: > > src/launcher/fetcher.cpp, lines 127-128 > > > > > > Why not just check for any 2xx code then? Aren't they all successful in > > one way or another? HTTP can return 2

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review92015 --- src/launcher/fetcher.cpp (lines 127 - 128)

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review91913 --- Ship it! src/launcher/fetcher.cpp (line 134)

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review91909 --- Patch looks great! Reviews applied: [36547] All tests passed. - M

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review91904 --- Ship it! Ship It! - Bernd Mathiske On July 16, 2015, 9:55 a.m.,

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/ --- (Updated July 16, 2015, 6:55 p.m.) Review request for mesos, Bernd Mathiske and

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review91892 --- Patch looks great! Reviews applied: [36547] All tests passed. - M

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review91891 --- src/launcher/fetcher.cpp (line 135)

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/ --- (Updated July 16, 2015, 2:48 p.m.) Review request for mesos, Bernd Mathiske and

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

> On July 16, 2015, 2:37 p.m., Joerg Schad wrote: > > src/launcher/fetcher.cpp, line 128 > > > > > > indentation +2 spaces > > (https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Conditionals) see lin

Re: Review Request 36547: Fixed fetcher failing for FTP URIs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36547/#review91885 --- src/launcher/fetcher.cpp (line 123)