Return-Path: X-Original-To: apmail-mesos-dev-archive@www.apache.org Delivered-To: apmail-mesos-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 624D61762F for ; Tue, 13 Jan 2015 18:46:22 +0000 (UTC) Received: (qmail 3193 invoked by uid 500); 13 Jan 2015 18:46:23 -0000 Delivered-To: apmail-mesos-dev-archive@mesos.apache.org Received: (qmail 3125 invoked by uid 500); 13 Jan 2015 18:46:23 -0000 Mailing-List: contact dev-help@mesos.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@mesos.apache.org Delivered-To: mailing list dev@mesos.apache.org Received: (qmail 3105 invoked by uid 99); 13 Jan 2015 18:46:23 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 13 Jan 2015 18:46:23 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 8C8601D321A; Tue, 13 Jan 2015 18:46:20 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============6033951717275854657==" MIME-Version: 1.0 Subject: Re: Review Request 29496: Added example hook module. From: "Kapil Arya" To: "Niklas Nielsen" , "mesos" , "Kapil Arya" Date: Tue, 13 Jan 2015 18:46:20 -0000 Message-ID: <20150113184620.23997.82438@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Kapil Arya" X-ReviewGroup: mesos X-ReviewRequest-URL: https://reviews.apache.org/r/29496/ X-Sender: "Kapil Arya" References: <20150106135534.26560.46083@reviews.apache.org> In-Reply-To: <20150106135534.26560.46083@reviews.apache.org> Reply-To: "Kapil Arya" X-ReviewRequest-Repository: mesos-git --===============6033951717275854657== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit > On Jan. 6, 2015, 8:55 a.m., Niklas Nielsen wrote: > > src/examples/test_hook_module.cpp, lines 38-40 > > > > > > I wonder if we can consolidate these strings with the ones in the test code. Have you tried to extern them from the test and grab them here? The strings are defined in two separate objects and it's not trivial to share them. Especially, when we want to test loading one object without necessarily using the other. However, I have now added a comment about the source and that these should be kept in sync. > On Jan. 6, 2015, 8:55 a.m., Niklas Nielsen wrote: > > src/tests/hook_tests.cpp, lines 117-118 > > > > > > I am not sure I understand why you are doing this. Can you expand a little bit? We need to read the taskInfo to verify the presence of labels inserted by the label decorator hook. - Kapil ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29496/#review66823 ----------------------------------------------------------- On Jan. 13, 2015, 1:37 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29496/ > ----------------------------------------------------------- > > (Updated Jan. 13, 2015, 1:37 p.m.) > > > Review request for mesos. > > > Bugs: MESOS-2060 > https://issues.apache.org/jira/browse/MESOS-2060 > > > Repository: mesos-git > > > Description > ------- > > This module provides hooks for master label decorator, slave executor environment decorator and slave remove executor. A couple of test cases are also provided to verify the hooks from this module. > > > Diffs > ----- > > src/Makefile.am fc0c3227466ccf364353a739fec8d9532ea3c683 > src/examples/test_hook_module.cpp PRE-CREATION > src/tests/hook_tests.cpp PRE-CREATION > src/tests/module.hpp bc1a37df7f95e945363418429fef4d090907c73a > src/tests/module.cpp 6cec1cbafe4750cff005191c51ce7c08149c18f4 > > Diff: https://reviews.apache.org/r/29496/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Kapil Arya > > --===============6033951717275854657==--