From reviews-return-27962-apmail-mesos-reviews-archive=mesos.apache.org@mesos.apache.org Sun Mar 20 03:57:49 2016 Return-Path: X-Original-To: apmail-mesos-reviews-archive@minotaur.apache.org Delivered-To: apmail-mesos-reviews-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 822DF1827C for ; Sun, 20 Mar 2016 03:57:49 +0000 (UTC) Received: (qmail 73686 invoked by uid 500); 20 Mar 2016 03:57:49 -0000 Delivered-To: apmail-mesos-reviews-archive@mesos.apache.org Received: (qmail 73658 invoked by uid 500); 20 Mar 2016 03:57:49 -0000 Mailing-List: contact reviews-help@mesos.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@mesos.apache.org Delivered-To: mailing list reviews@mesos.apache.org Received: (qmail 73644 invoked by uid 99); 20 Mar 2016 03:57:49 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 20 Mar 2016 03:57:49 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 5EA7D28B1C9; Sun, 20 Mar 2016 03:57:48 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============8829030174264295030==" MIME-Version: 1.0 Subject: Re: Review Request 45046: Created URI.filename to name fetched files in sandbox where appropriate. From: Mesos ReviewBot To: Zhitao Li , Vinod Kone Cc: Mesos ReviewBot , Michael Browning , mesos Date: Sun, 20 Mar 2016 03:57:48 -0000 Message-ID: <20160320035748.8674.49786@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: Mesos ReviewBot X-ReviewGroup: mesos X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/45046/ X-Sender: Mesos ReviewBot References: <20160319220358.8674.8616@reviews.apache.org> In-Reply-To: <20160319220358.8674.8616@reviews.apache.org> Reply-To: Mesos ReviewBot X-ReviewRequest-Repository: mesos --===============8829030174264295030== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- 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 > > --===============8829030174264295030==--