> On April 15, 2013, 10:59 p.m., Vinod Kone wrote: > > hadoop/hadoop-0.20.2-cdh3u3_hadoop-env.sh.patch, line 10 > > <https://reviews.apache.org/r/10489/diff/1/?file=281326#file281326line10> > > > > Cool. I'm assuming you tested this? > > Brenden Matthews wrote: > Works for me (does that count as testing?). > > Ben Mahler wrote: > Does the tutorial succeed locally? > > Brenden Matthews wrote: > It fails for me, but only because of how the build machine is set up. > I'll try to verify that it works on a vanilla machine.
let me know once you confirm, so that i can commit it. - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10489/#review19222 ----------------------------------------------------------- On April 15, 2013, 9:17 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10489/ > ----------------------------------------------------------- > > (Updated April 15, 2013, 9:17 p.m.) > > > Review request for mesos. > > > Description > ------- > > Fix Hadoop classpath for tutorial & deployment. > > > Diffs > ----- > > 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 > > Diff: https://reviews.apache.org/r/10489/diff/ > > > Testing > ------- > > Used in production at Airbnb. > > > Thanks, > > Brenden Matthews > >
