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




core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java
Lines 92 (patched)
<https://reviews.apache.org/r/63327/#comment267396>

    Unused import.



core/src/test/java/org/apache/oozie/test/XTestCase.java
Lines 1028-1030 (original), 1029-1031 (patched)
<https://reviews.apache.org/r/63327/#comment267398>

    These properties should be set in case of capacity scheduler used.


- Peter Cseh


On Nov. 6, 2017, 12:08 p.m., Peter Bacsko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/63327/
> -----------------------------------------------------------
> 
> (Updated Nov. 6, 2017, 12:08 p.m.)
> 
> 
> Review request for oozie, András Piros, Attila Sasvari, Peter Cseh, and 
> Robert Kanter.
> 
> 
> Repository: oozie-git
> 
> 
> Description
> -------
> 
> https://issues.apache.org/jira/browse/OOZIE-2897
> 
> 
> Diffs
> -----
> 
>   client/src/main/resources/oozie-common-1.0.xsd ddae91224 
>   core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java 
> 6a25e5c89 
>   core/src/main/java/org/apache/oozie/action/hadoop/YarnACLHandler.java 
> PRE-CREATION 
>   
> core/src/main/java/org/apache/oozie/workflow/lite/LauncherConfigHandler.java 
> c36774239 
>   core/src/main/resources/oozie-default.xml 8285df0a7 
>   
> core/src/test/java/org/apache/oozie/action/hadoop/ActionExecutorTestCase.java 
> 36b302667 
>   core/src/test/java/org/apache/oozie/action/hadoop/LauncherMainTester.java 
> 43c71b0ad 
>   
> core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java 
> adb2168ca 
>   
> core/src/test/java/org/apache/oozie/action/hadoop/TestLauncherConfigurationInjector.java
>  4c5b0f972 
>   core/src/test/java/org/apache/oozie/action/hadoop/TestYarnACLHandler.java 
> PRE-CREATION 
>   core/src/test/java/org/apache/oozie/service/TestSchemaService.java 
> 940868aeb 
>   core/src/test/java/org/apache/oozie/test/XTestCase.java 912c3f316 
>   
> core/src/test/java/org/apache/oozie/workflow/lite/TestLiteWorkflowAppParser.java
>  a361078ac 
>   core/src/test/resources/fair-scheduler-alloc.xml PRE-CREATION 
>   core/src/test/resources/wf-schema-global-launcherconf.xml 9cd4f6c37 
> 
> 
> Diff: https://reviews.apache.org/r/63327/diff/3/
> 
> 
> Testing
> -------
> 
> - Unit test added
> - Verified on a 4-node cluster
> 
> 
> Thanks,
> 
> Peter Bacsko
> 
>

Reply via email to