falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Srikanth Sundarrajan" <srik...@hotmail.com>
Subject Re: Review Request 19286: FALCON-356 Merge OozieProcessMapper and OozieProcessWorkflowBuilder
Date Tue, 18 Mar 2014 05:48:27 GMT

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



common/src/main/java/org/apache/falcon/util/ReflectionUtils.java
<https://reviews.apache.org/r/19286/#comment69089>

    Should constructor args be var-args instead ?



common/src/main/java/org/apache/falcon/util/ReflectionUtils.java
<https://reviews.apache.org/r/19286/#comment69090>

    What if arg is null ?



feed/src/main/java/org/apache/falcon/workflow/OozieFeedWorkflowBuilder.java
<https://reviews.apache.org/r/19286/#comment69091>

    Why does the log say process validity ? Isn't this feed workflow builder ?



feed/src/main/java/org/apache/falcon/workflow/OozieFeedWorkflowBuilder.java
<https://reviews.apache.org/r/19286/#comment69093>

    Dont we need to check for replicationCoord being null ? 



feed/src/main/java/org/apache/falcon/workflow/OozieFeedWorkflowBuilder.java
<https://reviews.apache.org/r/19286/#comment69094>

    Why are the retention mapper & replication mapper returning coord / workflows etc.
Can we keep their behavior consistent, it is quite useful in following the code 


- Srikanth Sundarrajan


On March 17, 2014, 9:50 a.m., shwethags wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19286/
> -----------------------------------------------------------
> 
> (Updated March 17, 2014, 9:50 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> There is one to one mapping between OozieProcessMapper and OozieProcessWorkflowBuilder.
Entity to oozie workflow mapping is spread across these two classes. Same goes for OozieFeedMapper
and OozieFeedWorkflowBuilder.
> 
> 	modified:   common/src/main/java/org/apache/falcon/util/ReflectionUtils.java
> 	modified:   common/src/main/java/org/apache/falcon/workflow/WorkflowBuilder.java
> 	renamed:    feed/src/main/java/org/apache/falcon/converter/OozieFeedMapper.java ->
feed/src/main/java/org/apache/falcon/workflow/OozieFeedWorkflowBuilder.java
> 	renamed:    feed/src/test/java/org/apache/falcon/converter/OozieFeedMapperTest.java
-> feed/src/test/java/org/apache/falcon/converter/OozieFeedWorkflowBuilderTest.java
> 	modified:   oozie/src/main/java/org/apache/falcon/util/OozieUtils.java
> 	renamed:    oozie/src/main/java/org/apache/falcon/converter/AbstractOozieEntityMapper.java
-> oozie/src/main/java/org/apache/falcon/workflow/OozieWorkflowBuilder.java
> 	modified:   oozie/src/main/java/org/apache/falcon/workflow/engine/OozieWorkflowEngine.java
> 	renamed:    process/src/main/java/org/apache/falcon/converter/OozieProcessMapper.java
-> process/src/main/java/org/apache/falcon/workflow/OozieProcessWorkflowBuilder.java
> 	deleted:    process/src/test/java/org/apache/falcon/converter/OozieProcessMapperLateProcessTest.java
> 	renamed:    process/src/test/java/org/apache/falcon/converter/OozieProcessMapperTest.java
-> process/src/test/java/org/apache/falcon/converter/OozieProcessWorkflowBuilderTest.java
> 	modified:   retention/src/test/java/org/apache/falcon/retention/FeedEvictorTest.java
> 
> 
> Diffs
> -----
> 
>   common/src/main/java/org/apache/falcon/util/ReflectionUtils.java 4a00fa9 
>   common/src/main/java/org/apache/falcon/workflow/WorkflowBuilder.java 26243e7 
>   feed/src/main/java/org/apache/falcon/converter/OozieFeedMapper.java 2b3315f 
>   feed/src/main/java/org/apache/falcon/workflow/OozieFeedWorkflowBuilder.java 5e3a30e

>   feed/src/test/java/org/apache/falcon/converter/OozieFeedMapperTest.java e610df2 
>   feed/src/test/java/org/apache/falcon/converter/OozieFeedWorkflowBuilderTest.java PRE-CREATION

>   oozie/src/main/java/org/apache/falcon/converter/AbstractOozieEntityMapper.java f443939

>   oozie/src/main/java/org/apache/falcon/util/OozieUtils.java 2f53370 
>   oozie/src/main/java/org/apache/falcon/workflow/OozieWorkflowBuilder.java e5a01ca 
>   oozie/src/main/java/org/apache/falcon/workflow/engine/OozieWorkflowEngine.java ac8862e

>   process/src/main/java/org/apache/falcon/converter/OozieProcessMapper.java e638961 
>   process/src/main/java/org/apache/falcon/workflow/OozieProcessWorkflowBuilder.java 4e5e8c6

>   process/src/test/java/org/apache/falcon/converter/OozieProcessMapperLateProcessTest.java
fbda0ea 
>   process/src/test/java/org/apache/falcon/converter/OozieProcessMapperTest.java 22bf9fe

>   process/src/test/java/org/apache/falcon/converter/OozieProcessWorkflowBuilderTest.java
PRE-CREATION 
>   retention/src/test/java/org/apache/falcon/retention/FeedEvictorTest.java 1e7cc04 
> 
> Diff: https://reviews.apache.org/r/19286/diff/
> 
> 
> Testing
> -------
> 
> UTs
> 
> 
> Thanks,
> 
> shwethags
> 
>


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