----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42246/#review114228 -----------------------------------------------------------
src/hdfs/hdfs.cpp (line 315) <https://reviews.apache.org/r/42246/#comment175043> According the document here https://docs.google.com/document/d/1p8tmQrGtxG6keZVo19gvHPr9WHxeny6PHTVofcLWqco/edit# We could have below cases in uri. ``` // mailto:[email protected] // news:comp.infosystems.www.servers.unix // tel:+1-816-555-1212 // telnet://192.0.2.16:80/ ``` But as I know, all protocol supported by HDFS client contains `://`, here also look good to me. - haosdent huang On Jan. 13, 2016, 10:07 a.m., Bernd Mathiske wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42246/ > ----------------------------------------------------------- > > (Updated Jan. 13, 2016, 10:07 a.m.) > > > Review request for mesos, haosdent huang, Jie Yu, and switched to 'mcypark'. > > > Bugs: MESOS-4304 > https://issues.apache.org/jira/browse/MESOS-4304 > > > Repository: mesos > > > Description > ------- > > Fixed support for non-HDFS URIs by Hadoop client. > > > Diffs > ----- > > src/hdfs/hdfs.cpp 51f016b049c6e947c4d27fbbbf79c53f9ec5a51e > > Diff: https://reviews.apache.org/r/42246/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Bernd Mathiske > >
