-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2769/#review5019
-----------------------------------------------------------


Couple of comments.


branches/3.1/distro/src/main/bin/addtowar.sh
<https://reviews.apache.org/r/2769/#comment11020>

    Can this variable be moved to the section where its used?



branches/3.1/distro/src/main/bin/addtowar.sh
<https://reviews.apache.org/r/2769/#comment11021>

    Is the version of Hadoop confirmed as 0.23.0? I thought 0.23.1 is the 
branch that we are certifying against.


- Santhosh


On 2012-02-10 04:52:29, Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/2769/
> -----------------------------------------------------------
> 
> (Updated 2012-02-10 04:52:29)
> 
> 
> Review request for oozie.
> 
> 
> Summary
> -------
> 
> This script injects required Jars into Oozie server's WEB-INF/lib directory.
> 
> For running oozie-setup.sh -hadoop <version> <hadoop_directory>
> hadoop_directory should point to the installation directory of hadoopcoretree 
> i.e. top level directory containing required hdfs,mapred and common jars.
> version = 0.23.0
> 
> Furthermore, all required env variables as specified on the hadoop 23 
> documentation should be set to be able to submit jobs
> 
> Some extra whitespaces removed.
> 
> 
> Diffs
> -----
> 
>   branches/3.1/distro/src/main/bin/addtowar.sh 1242192 
> 
> Diff: https://reviews.apache.org/r/2769/diff
> 
> 
> Testing
> -------
> 
> yes
> 
> 
> Thanks,
> 
> Mona
> 
>

Reply via email to