Re: Review Request 44239: Introduced copy fetcher plugin for file URIs.

2016-03-02 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44239/ --- (Updated March 3, 2016, 2:02 a.m.) Review request for mesos and Jie Yu.

Re: Review Request 44239: Introduced copy fetcher plugin for file URIs.

2016-03-02 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44239/ --- (Updated March 3, 2016, 1:47 a.m.) Review request for mesos and Jie Yu.

Re: Review Request 44239: Introduced copy fetcher plugin for file URIs.

2016-03-02 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44239/#review121758 --- Fix it, then Ship it! src/uri/fetchers/copy.cpp (line 85)

Re: Review Request 44239: Introduced copy fetcher plugin for file URIs.

2016-03-02 Thread Jojy Varghese
> On March 2, 2016, 3:26 p.m., Guangya Liu wrote: > > src/uri/fetchers/copy.cpp, line 72 > > > > > > Printing the `uri` in the failure may help improve the debug ability? The pattern is same as all other fetchers.

Re: Review Request 44239: Introduced copy fetcher plugin for file URIs.

2016-03-02 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44239/ --- (Updated March 2, 2016, 9:44 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 44239: Introduced copy fetcher plugin for file URIs.

2016-03-02 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44239/#review121655 --- src/uri/fetchers/copy.cpp (line 72)

Review Request 44239: Introduced copy fetcher plugin for file URIs.

2016-03-01 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44239/ --- Review request for mesos and Jie Yu. Bugs: MESOS-4834