You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by Karishma Gulati <gu...@yahoo.com> on 2015/02/19 08:46:54 UTC

Re: Review Request 31131: [Falcon-1034] Add test for FALCON-677 - wherein feed data and stats paths no longer default to /tmp/

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

(Updated Feb. 19, 2015, 7:46 a.m.)


Review request for Falcon.


Summary (updated)
-----------------

[Falcon-1034] Add test for FALCON-677 - wherein feed data and stats paths no longer default to /tmp/


Repository: falcon-git


Description
-------

Earlier if in a feed, the data and stats paths weren't mentioned, they'd default to /tmp/. Now, feed submission should fail if data path is not mentioned. More details at https://issues.apache.org/jira/browse/FALCON-677


Diffs (updated)
-----

  falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/Entities/FeedMerlin.java a458de9 
  falcon-regression/merlin/src/test/java/org/apache/falcon/regression/FeedSubmitTest.java 374c442 

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


Testing
-------

Added changes to FeedSubmitTest


Thanks,

Karishma Gulati


Re: Review Request 31131: [Falcon-1034] Add test for FALCON-677 - wherein feed data and stats paths no longer default to /tmp/

Posted by samarth gupta <sa...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31131/#review73518
-----------------------------------------------------------

Ship it!


Ship It!

- samarth gupta


On Feb. 19, 2015, 7:46 a.m., Karishma Gulati wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31131/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2015, 7:46 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Earlier if in a feed, the data and stats paths weren't mentioned, they'd default to /tmp/. Now, feed submission should fail if data path is not mentioned. More details at https://issues.apache.org/jira/browse/FALCON-677
> 
> 
> Diffs
> -----
> 
>   falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/Entities/FeedMerlin.java a458de9 
>   falcon-regression/merlin/src/test/java/org/apache/falcon/regression/FeedSubmitTest.java 374c442 
> 
> Diff: https://reviews.apache.org/r/31131/diff/
> 
> 
> Testing
> -------
> 
> Added changes to FeedSubmitTest
> 
> 
> Thanks,
> 
> Karishma Gulati
> 
>