You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by Paul Isaychuk <pi...@hortonworks.com> on 2014/08/18 16:03:26 UTC

Review Request 24799: [FALCON-610] Refactoring, documentation stuff

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

Review request for Falcon.


Repository: falcon-git


Description
-------

refactoring, cleanup, documentation


Diffs
-----

  merlin-core/src/main/java/org/apache/falcon/regression/core/util/HadoopUtil.java c33700c 
  merlin-core/src/main/java/org/apache/falcon/regression/core/util/TimeUtil.java 15a876f 
  merlin/src/test/java/org/apache/falcon/regression/EmbeddedPigScriptTest.java bf0e8d3 
  merlin/src/test/java/org/apache/falcon/regression/FeedResumeTest.java 0af3a4e 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceKillsTest.java 515b18e 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceRerunTest.java 8b40cf1 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceResumeTest.java 6bcb24a 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceRunningTest.java 559df48 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceStatusTest.java 86f7a86 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java e77d534 
  merlin/src/test/java/org/apache/falcon/regression/hcat/HCatReplicationTest.java 860e680 
  merlin/src/test/java/org/apache/falcon/regression/prism/PrismConcurrentRequestTest.java d4ba5b0 
  merlin/src/test/java/org/apache/falcon/regression/prism/RetentionTest.java c8cef02 

Diff: https://reviews.apache.org/r/24799/diff/


Testing
-------

Tested


Thanks,

Paul Isaychuk


Re: Review Request 24799: [FALCON-610] Refactoring, documentation stuff

Posted by Raghav Gautam <ra...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24799/#review51005
-----------------------------------------------------------

Ship it!


Ship It!

- Raghav Gautam


On Aug. 18, 2014, 7:06 a.m., Paul Isaychuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/24799/
> -----------------------------------------------------------
> 
> (Updated Aug. 18, 2014, 7:06 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> refactoring, cleanup, documentation
> 
> 
> Diffs
> -----
> 
>   merlin-core/src/main/java/org/apache/falcon/regression/core/util/HadoopUtil.java c33700c 
>   merlin-core/src/main/java/org/apache/falcon/regression/core/util/TimeUtil.java 15a876f 
>   merlin/src/test/java/org/apache/falcon/regression/EmbeddedPigScriptTest.java bf0e8d3 
>   merlin/src/test/java/org/apache/falcon/regression/FeedResumeTest.java 0af3a4e 
>   merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceKillsTest.java 515b18e 
>   merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceRerunTest.java 8b40cf1 
>   merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceResumeTest.java 6bcb24a 
>   merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceRunningTest.java 559df48 
>   merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceStatusTest.java 86f7a86 
>   merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java e77d534 
>   merlin/src/test/java/org/apache/falcon/regression/hcat/HCatReplicationTest.java 860e680 
>   merlin/src/test/java/org/apache/falcon/regression/prism/PrismConcurrentRequestTest.java d4ba5b0 
>   merlin/src/test/java/org/apache/falcon/regression/prism/RetentionTest.java c8cef02 
> 
> Diff: https://reviews.apache.org/r/24799/diff/
> 
> 
> Testing
> -------
> 
> Tested
> 
> 
> Thanks,
> 
> Paul Isaychuk
> 
>


Re: Review Request 24799: [FALCON-610] Refactoring, documentation stuff

Posted by Paul Isaychuk <pi...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24799/
-----------------------------------------------------------

(Updated Aug. 18, 2014, 2:06 p.m.)


Review request for Falcon.


Repository: falcon-git


Description
-------

refactoring, cleanup, documentation


Diffs
-----

  merlin-core/src/main/java/org/apache/falcon/regression/core/util/HadoopUtil.java c33700c 
  merlin-core/src/main/java/org/apache/falcon/regression/core/util/TimeUtil.java 15a876f 
  merlin/src/test/java/org/apache/falcon/regression/EmbeddedPigScriptTest.java bf0e8d3 
  merlin/src/test/java/org/apache/falcon/regression/FeedResumeTest.java 0af3a4e 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceKillsTest.java 515b18e 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceRerunTest.java 8b40cf1 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceResumeTest.java 6bcb24a 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceRunningTest.java 559df48 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceStatusTest.java 86f7a86 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java e77d534 
  merlin/src/test/java/org/apache/falcon/regression/hcat/HCatReplicationTest.java 860e680 
  merlin/src/test/java/org/apache/falcon/regression/prism/PrismConcurrentRequestTest.java d4ba5b0 
  merlin/src/test/java/org/apache/falcon/regression/prism/RetentionTest.java c8cef02 

Diff: https://reviews.apache.org/r/24799/diff/


Testing
-------

Tested


Thanks,

Paul Isaychuk


Re: Review Request 24799: [FALCON-610] Refactoring, documentation stuff

Posted by Paul Isaychuk <pi...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24799/
-----------------------------------------------------------

(Updated Aug. 18, 2014, 2:06 p.m.)


Review request for Falcon.


Repository: falcon-git


Description
-------

refactoring, cleanup, documentation


Diffs
-----

  merlin-core/src/main/java/org/apache/falcon/regression/core/util/HadoopUtil.java c33700c 
  merlin-core/src/main/java/org/apache/falcon/regression/core/util/TimeUtil.java 15a876f 
  merlin/src/test/java/org/apache/falcon/regression/EmbeddedPigScriptTest.java bf0e8d3 
  merlin/src/test/java/org/apache/falcon/regression/FeedResumeTest.java 0af3a4e 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceKillsTest.java 515b18e 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceRerunTest.java 8b40cf1 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceResumeTest.java 6bcb24a 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceRunningTest.java 559df48 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceStatusTest.java 86f7a86 
  merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java e77d534 
  merlin/src/test/java/org/apache/falcon/regression/hcat/HCatReplicationTest.java 860e680 
  merlin/src/test/java/org/apache/falcon/regression/prism/PrismConcurrentRequestTest.java d4ba5b0 
  merlin/src/test/java/org/apache/falcon/regression/prism/RetentionTest.java c8cef02 

Diff: https://reviews.apache.org/r/24799/diff/


Testing
-------

Tested


Thanks,

Paul Isaychuk