You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by Ruslan Ostafiychuk <ro...@hortonworks.com> on 2015/01/06 12:14:22 UTC

Re: Review Request 25336: [Falcon-671] Add a test in falcon-regression where one oozie workflow has various actions like mr, hive, pig

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



falcon-regression/merlin/src/test/resources/combinedActions/cluster-0.1.xml
<https://reviews.apache.org/r/25336/#comment110452>

    Why this was changed back to 'ivory' in 7th revision? It will be replaced to MerlinConstants.STAGING(WORKING)LOCATION anyway. We should avoid 'ivory' after FALCON-930


- Ruslan Ostafiychuk


On Dec. 30, 2014, 11:16 a.m., Karishma Gulati wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25336/
> -----------------------------------------------------------
> 
> (Updated Dec. 30, 2014, 11:16 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 6e25a60 
>   falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/ExecUtil.java 82c8919 
>   falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/HCatUtil.java d878ecb 
>   falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/OSUtil.java ed29d07 
>   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/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
> 
>