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



trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java
<https://reviews.apache.org/r/2059/#comment13756>

    This requirement came up when Mayank tested for multiple NN (federation) 
support. Without that , multiple NN access was not working. I remember Sanjay 
Radia proposed to follow this approach to get the URI.
    
    However it was a very long time back.
    


- Mohammad


On 2012-03-22 17:27:37, Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/2059/
> -----------------------------------------------------------
> 
> (Updated 2012-03-22 17:27:37)
> 
> 
> Review request for oozie, Mohammad Islam and Angelo K. Huang.
> 
> 
> Summary
> -------
> 
> This patch was originally worked on by Mayank Bansal. 
> I (Mona Chitnis) am in charge of handling this review request.
> 
> Description - adding hadoop.next support for oozie
> 
> 
> This addresses bug OOZIE-477.
>     https://issues.apache.org/jira/browse/OOZIE-477
> 
> 
> Diffs
> -----
> 
>   
> trunk/core/src/main/java/org/apache/oozie/action/hadoop/MapReduceActionExecutor.java
>  1303609 
>   trunk/client/src/main/java/org/apache/oozie/client/XOozieClient.java 
> 1303609 
>   
> trunk/core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java 
> 1303609 
>   
> trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java
>  1303609 
>   trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java 
> 1303609 
>   trunk/core/src/main/java/org/apache/oozie/service/WorkflowAppService.java 
> 1303609 
>   trunk/core/src/main/resources/oozie-default.xml 1303609 
>   
> trunk/core/src/test/java/org/apache/oozie/service/TestLiteWorkflowAppService.java
>  1303609 
> 
> Diff: https://reviews.apache.org/r/2059/diff
> 
> 
> Testing
> -------
> 
> tested against current unit tests
> 
> 
> Thanks,
> 
> Mona
> 
>

Reply via email to