> On April 8, 2013, 9:55 p.m., Vinod Kone wrote: > > hadoop/TUTORIAL.sh, line 547 > > <https://reviews.apache.org/r/10346/diff/1/?file=279031#file279031line547> > > > > Is this required by the pre-package script? If yes, please add a > > comment here. > > Brenden Matthews wrote: > In my case I use it, but it's not required. I use the hook scripts for > doing things like including custom configuration parameters (i.e., 'cp > /etc/hadoop/config/* config/') before the package is created, and copying the > package to HDFS after it's ready. > > Vinod Kone wrote: > Just so that I understand, is the 'pre-package' and 'post-package' > scripts specific to airbnb or are they part of cloudera distributions. If its > the former, I would be skeptical of adding it to the tutorial script.
These are optional hook scripts that are specific to airbnb. I need a way of adding specific configuration parameters to the hadoop dist, and I think this is the best way to go about it. The scripts are not required and the tutorial will function with or without them. - Brenden ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10346/#review18797 ----------------------------------------------------------- On April 8, 2013, 9:21 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10346/ > ----------------------------------------------------------- > > (Updated April 8, 2013, 9:21 p.m.) > > > Review request for mesos. > > > Description > ------- > > - Added CDH4.2.0 support to TUTORIAL.sh > - Use .tar.xz for Hadoop dist (LZMA) > - Fixed Hadoop dist extraction in launcher > - Add pre/post packaging hooks to TUTORIAL.sh > - Don't run the jobtracker at the end by default > > > Diffs > ----- > > hadoop/Makefile.am bd8919a > hadoop/TUTORIAL.sh b0f77f3 > hadoop/hadoop-2.0.0-mr1-cdh4.1.2_hadoop-env.sh.patch c61ccec > hadoop/hadoop-2.0.0-mr1-cdh4.2.0_hadoop-env.sh.patch PRE-CREATION > hadoop/hadoop-2.0.0-mr1-cdh4.2.0_mesos.patch PRE-CREATION > src/launcher/launcher.cpp eb8d6fb > > Diff: https://reviews.apache.org/r/10346/diff/ > > > Testing > ------- > > Works in production on Hadoop at airbnb. > > > Thanks, > > Brenden Matthews > >
