falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ruslan Ostafiychuk" <rostafiyc...@hortonworks.com>
Subject Re: Review Request 25336: [Falcon-671] Add a test in falcon-regression where one oozie workflow has various actions like mr, hive, pig
Date Tue, 09 Sep 2014 12:25:48 GMT

-----------------------------------------------------------
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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message