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

Ship it!


+1 looks good

- Mayank


On 2012-03-06 05:39:54, Alejandro Abdelnur wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/4186/
> -----------------------------------------------------------
> 
> (Updated 2012-03-06 05:39:54)
> 
> 
> Review request for oozie.
> 
> 
> Summary
> -------
> 
> hadoop.configurations= [<AUTHORITY>=<CONFIG_FILE>]* 
> 
> Where <AUTHORITY> is the HOST:PORT of the Hadoop JT/NN and <CONFIG_FILE> is 
> the Hadoop configuration file to load from the local filesystem with the 
> defaults for the corresponding JT/NN. 
> 
> In addition, a '*' wildcard would be support for the <AUTHORITY>, this would 
> be used as default if the job used authority does not have a custom config. 
> 
> The Hadoop configuration file would be looked in the Oozie conf/ directory 
> 
> An example of this configuration would be: 
> 
> hadoop.configuration= *=hadoop-defaultconf.xml 
> myjobtracker:8021=hadoop-cluster1.xml, mynamenode:8020=hadoop-cluster1.xml 
> 
> Documentation is still missing, coming soon.
> 
> 
> This addresses bug OOZIE-736.
>     https://issues.apache.org/jira/browse/OOZIE-736
> 
> 
> Diffs
> -----
> 
>   trunk/core/src/main/conf/hadoop-config.xml PRE-CREATION 
>   
> trunk/core/src/main/java/org/apache/oozie/service/HadoopAccessorService.java 
> 1297338 
>   trunk/core/src/main/resources/oozie-default.xml 1297338 
>   
> trunk/core/src/test/java/org/apache/oozie/service/TestHadoopAccessorService.java
>  1297338 
>   trunk/core/src/test/java/org/apache/oozie/test/XFsTestCase.java 1297338 
>   trunk/core/src/test/java/org/apache/oozie/test/XTestCase.java 1297338 
>   trunk/core/src/test/resources/hadoop-config.xml PRE-CREATION 
>   trunk/core/src/test/resources/test-hadoop-config.xml PRE-CREATION 
>   trunk/docs/src/site/twiki/AG_HadoopConfiguration.twiki PRE-CREATION 
>   trunk/docs/src/site/twiki/DG_UsingHadoopKerberos.twiki 1297338 
>   trunk/docs/src/site/twiki/index.twiki 1297338 
> 
> Diff: https://reviews.apache.org/r/4186/diff
> 
> 
> Testing
> -------
> 
> run all testcases with Hadoop 1
> 
> 
> Thanks,
> 
> Alejandro
> 
>

Reply via email to