Re: Review Request 42314: Allow schemes in HDFS URI fetcher plugin to be configurable.

2016-01-14 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42314/#review114658 --- Patch looks great! Reviews applied: [42314] Passed command: expor

Re: Review Request 42314: Allow schemes in HDFS URI fetcher plugin to be configurable.

2016-01-14 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42314/ --- (Updated Jan. 15, 2016, 2:31 a.m.) Review request for mesos, Adam B and Jie Yu.

Re: Review Request 42314: Allow schemes in HDFS URI fetcher plugin to be configurable.

2016-01-14 Thread haosdent huang
> On Jan. 14, 2016, 11:35 p.m., Timothy Chen wrote: > > src/uri/fetchers/hadoop.cpp, line 59 > > > > > > This seems unncessary, even if there is duplicate I think we simply > > just override the entry when we registe

Re: Review Request 42314: Allow schemes in HDFS URI fetcher plugin to be configurable.

2016-01-14 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42314/#review114606 --- src/uri/fetchers/hadoop.cpp (line 59)

Re: Review Request 42314: Allow schemes in HDFS URI fetcher plugin to be configurable.

2016-01-14 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42314/ --- (Updated Jan. 14, 2016, 7:09 p.m.) Review request for mesos, Adam B and Jie Yu.

Re: Review Request 42314: Allow schemes in HDFS URI fetcher plugin to be configurable.

2016-01-14 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42314/ --- (Updated Jan. 14, 2016, 7:06 p.m.) Review request for mesos, Adam B and Jie Yu.

Re: Review Request 42314: Allow schemes in HDFS URI fetcher plugin to be configurable.

2016-01-14 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42314/ --- (Updated Jan. 14, 2016, 6:59 p.m.) Review request for mesos, Adam B and Jie Yu.

Review Request 42314: Allow schemes in HDFS URI fetcher plugin to be configurable.

2016-01-14 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42314/ --- Review request for mesos. Repository: mesos Description --- Allow scheme