----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10489/#review19286 -----------------------------------------------------------
hadoop/TUTORIAL.sh <https://reviews.apache.org/r/10489/#comment39953> So this is not the default? I wonder how it worked for us before? hadoop/hadoop-2.0.0-mr1-cdh4.1.2_hadoop-env.sh.patch <https://reviews.apache.org/r/10489/#comment39952> hmm..so you need to explicitly specify them for cdh4 but not for cdh3 and apache hadoop? thats confusing. i would suggest reverting these changes in that case. - Vinod Kone On April 16, 2013, 9:59 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10489/ > ----------------------------------------------------------- > > (Updated April 16, 2013, 9:59 p.m.) > > > Review request for mesos. > > > Description > ------- > > Fix Hadoop classpath for tutorial & deployment. > > > Diffs > ----- > > hadoop/TUTORIAL.sh f8131cd > hadoop/hadoop-0.20.2-cdh3u3_hadoop-env.sh.patch fb8b06b > hadoop/hadoop-0.20.205.0_hadoop-env.sh.patch 8f67ff9 > hadoop/hadoop-2.0.0-mr1-cdh4.1.2_hadoop-env.sh.patch 6d31578 > hadoop/hadoop-2.0.0-mr1-cdh4.2.0_hadoop-env.sh.patch 908ae4a > > Diff: https://reviews.apache.org/r/10489/diff/ > > > Testing > ------- > > Used in production at Airbnb. > > > Thanks, > > Brenden Matthews > >
