falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From PRAGYA MITTAL <mittal.pragy...@gmail.com>
Subject Re: Review Request 42795: [FALCON-1777] Add regression for HDFS replication (recipe)
Date Tue, 26 Jan 2016 14:49:49 GMT

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




falcon-regression/merlin/src/test/java/org/apache/falcon/regression/hive/dr/HdfsRecipeTest.java
(line 71)
<https://reviews.apache.org/r/42795/#comment177398>

    This can be @BeforeMethod



falcon-regression/merlin/src/test/java/org/apache/falcon/regression/hive/dr/HdfsRecipeTest.java
(line 103)
<https://reviews.apache.org/r/42795/#comment177397>

    Can we not wait for instance to go directly into SUCCEEDED state instead of RUNNING and
the SUCCEEDED. Any specific reason for doing so ?


- PRAGYA MITTAL


On Jan. 26, 2016, 11:45 a.m., Paul Isaychuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42795/
> -----------------------------------------------------------
> 
> (Updated Jan. 26, 2016, 11:45 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1777
>     https://issues.apache.org/jira/browse/FALCON-1777
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Add regression for HDFS replication (recipe)
> 
> 
> Diffs
> -----
> 
>   falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/Entities/RecipeMerlin.java
40fec08 
>   falcon-regression/merlin/src/test/java/org/apache/falcon/regression/hive/dr/HdfsRecipeTest.java
PRE-CREATION 
>   falcon-regression/merlin/src/test/resources/HdfsRecipe/hive-disaster-recovery-workflow.xml
7c4c53a 
>   falcon-regression/merlin/src/test/resources/HdfsRecipe/hive-disaster-recovery.properties
7c95db7 
> 
> Diff: https://reviews.apache.org/r/42795/diff/
> 
> 
> Testing
> -------
> 
> done
> 
> 
> Thanks,
> 
> Paul Isaychuk
> 
>


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