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



falcon-regression/merlin/src/test/java/org/apache/falcon/regression/CombinedActionsTest.java
<https://reviews.apache.org/r/25336/#comment91693>

    cannot be compiled after FALCON-675



falcon-regression/merlin/src/test/java/org/apache/falcon/regression/CombinedActionsTest.java
<https://reviews.apache.org/r/25336/#comment91692>

    Should be moved to HCatUtils. HCatReplicationTest and HCatProcessTest has 
it too.



falcon-regression/merlin/src/test/java/org/apache/falcon/regression/CombinedActionsTest.java
<https://reviews.apache.org/r/25336/#comment91691>

    TimeUtil.getMinuteDatesOnEitherSide(String, String, int, DateTimeFormatter) 
can be used


- Ruslan Ostafiychuk


On Sept. 5, 2014, 10:14 a.m., Karishma Gulati wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25336/
> -----------------------------------------------------------
> 
> (Updated Sept. 5, 2014, 10:14 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> A test needs to be added where one worflow has various possible actions like 
> mr, hive, pig. So far, we just specify the engine type as hive or pig for 
> such specific tests. 
> 
> 
> Diffs
> -----
> 
>   
> falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/BundleUtil.java
>  1f73523 
>   
> falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/OSUtil.java
>  86d4d47 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/CombinedActionsTest.java
>  PRE-CREATION 
>   falcon-regression/merlin/src/test/resources/combinedActions/cluster-0.1.xml 
> PRE-CREATION 
>   
> falcon-regression/merlin/src/test/resources/combinedActions/feed-template1.xml
>  PRE-CREATION 
>   
> falcon-regression/merlin/src/test/resources/combinedActions/feed-template1_hcat.xml
>  PRE-CREATION 
>   
> falcon-regression/merlin/src/test/resources/combinedActions/feed-template2.xml
>  PRE-CREATION 
>   
> falcon-regression/merlin/src/test/resources/combinedActions/feed-template2_hcat.xml
>  PRE-CREATION 
>   
> falcon-regression/merlin/src/test/resources/combinedActions/feed-template3.xml
>  PRE-CREATION 
>   falcon-regression/merlin/src/test/resources/combinedActions/process-agg.xml 
> PRE-CREATION 
>   falcon-regression/merlin/src/test/resources/combinedWorkflow/id.pig 
> PRE-CREATION 
>   
> falcon-regression/merlin/src/test/resources/combinedWorkflow/lib/oozie-examples-3.1.5.jar
>  PRE-CREATION 
>   falcon-regression/merlin/src/test/resources/combinedWorkflow/script.hql 
> PRE-CREATION 
>   falcon-regression/merlin/src/test/resources/combinedWorkflow/workflow.xml 
> PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/25336/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Karishma Gulati
> 
>

Reply via email to