Re: Review Request 66651: Supported host and port in hdfs constructor.

2018-06-06 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66651/ --- (Updated June 6, 2018, 1:42 a.m.) Review request for mesos, Jie Yu and Qian

Re: Review Request 66651: Supported host and port in hdfs constructor.

2018-06-06 Thread Gilbert Song
> On April 18, 2018, 5:51 a.m., Qian Zhang wrote: > > Can you add a test in `uri_fetcher_tests.cpp` to verify fetching a remote > > file via Hadoop fetcher plugin? for now, it is hard to find a long existing hdfs:// source to verify network connection reliability. I would say simulate in unit

Re: Review Request 66651: Supported host and port in hdfs constructor.

2018-04-18 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66651/#review201415 --- Can you add a test in `uri_fetcher_tests.cpp` to verify fetching

Re: Review Request 66651: Supported host and port in hdfs constructor.

2018-04-18 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66651/#review201407 --- Ship it! Ship It! - Qian Zhang On April 17, 2018, 1:32

Review Request 66651: Supported host and port in hdfs constructor.

2018-04-16 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66651/ --- Review request for mesos, Jie Yu and Qian Zhang. Bugs: MESOS-8794