----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11109/#review20659 -----------------------------------------------------------
hadoop/TUTORIAL.sh <https://reviews.apache.org/r/11109/#comment42652> The problem really lies here. The tutorial should aim to explain how to build a "Mesos distribution" and that should be articulated here. In other words, this is not about "building the Mesos executor package" but it's about "creating the Mesos distribution". The comment should probably also explain that the Mesos distribution is used to run the TaskTrackers via the "mapred.mesos.executor" configuration property. In fact, given that we have "mapred.mesos.executor" as a configuration property there is no reason not to be explicit here and call the final package 'hadoop-${distribution}-mesos.tar.gz'. Let's be explicit as much as we can. - Benjamin Hindman On May 14, 2013, 5:57 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11109/ > ----------------------------------------------------------- > > (Updated May 14, 2013, 5:57 p.m.) > > > Review request for mesos. > > > Description > ------- > > From e38084684887f59bce5694bb3457a94b2da026e3 Mon Sep 17 00:00:00 2001 > From: Brenden Matthews <[email protected]> > Date: Mon, 15 Apr 2013 14:08:01 -0700 > Subject: [PATCH 02/24] Run Hadoop tutorial binaries from within build dir. > > Review: https://reviews.apache.org/r/11109 > --- > hadoop/TUTORIAL.sh | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > > Diffs > ----- > > hadoop/TUTORIAL.sh d81c55562d217e8784745d4a8cedf78ada9f5453 > > Diff: https://reviews.apache.org/r/11109/diff/ > > > Testing > ------- > > Used in production at airbnb. > > > Thanks, > > Brenden Matthews > >
