You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Megha Sharma <ms...@apple.com> on 2017/01/10 00:53:06 UTC

Re: Review Request 55362: Support asynchronous fetching of URIs.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55362/
-----------------------------------------------------------

(Updated Jan. 10, 2017, 12:53 a.m.)


Review request for mesos and Jiang Yan Xu.


Bugs: MESOS-6601
    https://issues.apache.org/jira/browse/MESOS-6601


Repository: mesos


Description
-------

Mesos fetcher fetches the URIs specified in CommandInfo serially,
one after the other. If the task has too many URIs to be fetched
then it delays the execution of the task.


Diffs (updated)
-----

  src/launcher/fetcher.cpp 4456c28139966e42859cc6d2c79a1d90e83fb22f 

Diff: https://reviews.apache.org/r/55362/diff/


Testing
-------

make check on macOS.


Thanks,

Megha Sharma


Re: Review Request 55362: Support asynchronous fetching of URIs.

Posted by Mesos ReviewBot <re...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55362/#review161031
-----------------------------------------------------------



Patch looks great!

Reviews applied: [55362]

Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh

- Mesos ReviewBot


On Jan. 10, 2017, 12:53 a.m., Megha Sharma wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/55362/
> -----------------------------------------------------------
> 
> (Updated Jan. 10, 2017, 12:53 a.m.)
> 
> 
> Review request for mesos and Jiang Yan Xu.
> 
> 
> Bugs: MESOS-6601
>     https://issues.apache.org/jira/browse/MESOS-6601
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Mesos fetcher fetches the URIs specified in CommandInfo serially,
> one after the other. If the task has too many URIs to be fetched
> then it delays the execution of the task.
> 
> 
> Diffs
> -----
> 
>   src/launcher/fetcher.cpp 4456c28139966e42859cc6d2c79a1d90e83fb22f 
> 
> Diff: https://reviews.apache.org/r/55362/diff/
> 
> 
> Testing
> -------
> 
> make check on macOS.
> 
> 
> Thanks,
> 
> Megha Sharma
> 
>


Re: Review Request 55362: Support asynchronous fetching of URIs.

Posted by Megha Sharma <ms...@apple.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55362/
-----------------------------------------------------------

(Updated May 2, 2017, 12:08 a.m.)


Review request for mesos and Jiang Yan Xu.


Bugs: MESOS-6601
    https://issues.apache.org/jira/browse/MESOS-6601


Repository: mesos


Description
-------

Mesos fetcher fetches the URIs specified in CommandInfo serially,
one after the other. If the task has too many URIs to be fetched
then it delays the execution of the task.


Diffs (updated)
-----

  src/launcher/fetcher.cpp 42980f5a4a40b72f754156469e9fe60a952d1d87 


Diff: https://reviews.apache.org/r/55362/diff/3/

Changes: https://reviews.apache.org/r/55362/diff/2-3/


Testing
-------

make check on macOS.


Thanks,

Megha Sharma