----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45046/#review124436 -----------------------------------------------------------
Patch looks great! Reviews applied: [45046] Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On March 19, 2016, 10:03 p.m., Michael Browning wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45046/ > ----------------------------------------------------------- > > (Updated March 19, 2016, 10:03 p.m.) > > > Review request for mesos, Vinod Kone and Zhitao Li. > > > Bugs: MESOS-4735 > https://issues.apache.org/jira/browse/MESOS-4735 > > > Repository: mesos > > > Description > ------- > > Created URI.filename to name fetched files in sandbox where appropriate. > > > Diffs > ----- > > docs/fetcher.md f70939d8410516c9387a8cba86b5b75539a5fe9a > include/mesos/mesos.proto deb9c0910a27afd67276f54b3f666a878212727b > include/mesos/v1/mesos.proto a981e750c24cfc48177bbc9ca56f0c3ecfae1a1b > src/launcher/fetcher.cpp f85b118fb19cf9d4563f89847a783be35067e815 > src/slave/containerizer/fetcher.cpp > 33dfcade6beb53a5a6dbc41a8f3380f5cb30a161 > src/tests/fetcher_cache_tests.cpp 645dae208cb2b0aa2d2181d96eb1fd8893975430 > src/tests/fetcher_tests.cpp fb47706eb90ae5808bafe13c681d609a808b0c8e > > Diff: https://reviews.apache.org/r/45046/diff/ > > > Testing > ------- > > There are two paths by which a file gets fetched to the executor sandbox: the > without-cache path, where the fetcher downloads the file directly from the > specified URI, and the with-cache path, where it copies it from the cache. In > both cases, we verify that the file is saved to the sandbox directory with > the name specified by the "filename" field in the CommandInfo.URI proto. > > > Thanks, > > Michael Browning > >
