----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45046/#review124393 -----------------------------------------------------------
src/launcher/fetcher.cpp (lines 255 - 256) <https://reviews.apache.org/r/45046/#comment186915> a blank line here and ditto for the following src/tests/fetcher_cache_tests.cpp (line 669) <https://reviews.apache.org/r/45046/#comment186918> const? src/tests/fetcher_cache_tests.cpp (lines 690 - 691) <https://reviews.apache.org/r/45046/#comment186916> blank line here src/tests/fetcher_tests.cpp (line 654) <https://reviews.apache.org/r/45046/#comment186920> const? src/tests/fetcher_tests.cpp (line 693) <https://reviews.apache.org/r/45046/#comment186921> const src/tests/fetcher_tests.cpp (line 711) <https://reviews.apache.org/r/45046/#comment186922> const - Guangya Liu On 三月 18, 2016, 11:30 p.m., Michael Browning wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45046/ > ----------------------------------------------------------- > > (Updated 三月 18, 2016, 11:30 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 > >
