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

Ship it!


Ship It!

- Ruslan Ostafiychuk


On March 25, 2015, 2:18 p.m., Paul Isaychuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32307/
> -----------------------------------------------------------
> 
> (Updated March 25, 2015, 2:18 p.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1112
>     https://issues.apache.org/jira/browse/FALCON-1112
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Migrate methods related to *Merlin.java classes from Util.java to their 
> respective *Merlin.java
> 
> 
> Diffs
> -----
> 
>   
> falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/Entities/ClusterMerlin.java
>  22d8ca4 
>   
> falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/Entities/FeedMerlin.java
>  a458de9 
>   
> falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/Entities/ProcessMerlin.java
>  f869844 
>   
> falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/bundle/Bundle.java
>  876ffa4 
>   
> falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/helpers/entity/ProcessEntityHelper.java
>  57f2320 
>   
> falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/BundleUtil.java
>  c53d927 
>   
> falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/Util.java
>  e313eee 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/AuthorizationTest.java
>  0b934dc 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ELExpCurrentAndLastWeekTest.java
>  7cce198 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ELValidationsTest.java
>  18efe9b 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/EmbeddedPigScriptTest.java
>  bddf4ef 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/NewRetryTest.java
>  a666796 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceColoMixedTest.java
>  2fc4779 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java
>  588355a 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessLateRerunTest.java
>  338d90a 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/hcat/HCatProcessTest.java
>  d23d666 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/lineage/LineageApiProcessInstanceTest.java
>  2ec2b20 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/EntityDryRunTest.java
>  e87579f 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/FeedDelayParallelTimeoutTest.java
>  a829bd1 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/NewPrismProcessUpdateTest.java
>  3a5d71b 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/OptionalInputTest.java
>  95de483 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismFeedReplicationUpdateTest.java
>  9b8e9d3 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismFeedUpdateTest.java
>  7910fd7 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismSubmitTest.java
>  508cc08 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/ProcessPartitionExpVariableTest.java
>  a716b83 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/RescheduleKilledProcessTest.java
>  a363120 
>   
> falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/UpdateAtSpecificTimeTest.java
>  e578809 
> 
> Diff: https://reviews.apache.org/r/32307/diff/
> 
> 
> Testing
> -------
> 
> compilation goes through, no checkstyle issues, brief tests passed on rerun
> 
> 
> Thanks,
> 
> Paul Isaychuk
> 
>

Reply via email to