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 42854: [FALCON-1784] Add regression test for for FALCON-1647
Date Thu, 04 Feb 2016 05:41:21 GMT

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




falcon-regression/merlin/src/test/java/org/apache/falcon/regression/FeedSubmitAndScheduleTest.java
(line 172)
<https://reviews.apache.org/r/42854/#comment179042>

    Are the feeds getting deleted after the test passees?
    I have noticed so far that when we pass the names explicitly cleanUp does not handle deleting
such entities. Please check once.


- PRAGYA MITTAL


On Jan. 27, 2016, 3:53 p.m., Paul Isaychuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42854/
> -----------------------------------------------------------
> 
> (Updated Jan. 27, 2016, 3:53 p.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1784
>     https://issues.apache.org/jira/browse/FALCON-1784
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Falcon-1647 fixes the following issue.
> Create cluster entity as user1. Submit and schedule feed entity feed1 in this cluster
as user1. Now try to submit and schedule a feed entity feed2 in this cluster as user2. This
failed with error
> 
> Permission denied: user=user2, access=WRITE, inode="/apps/falcon/backupCluster/staging/falcon/workflows/feed":falcon:hdfs:drwxr-xr-x
> 
> This issue was fixed by Falcon-1647 and we should have a regression test to verify the
fix.
> 
> 
> Diffs
> -----
> 
>   falcon-regression/merlin/src/test/java/org/apache/falcon/regression/FeedSubmitAndScheduleTest.java
46015dc 
> 
> Diff: https://reviews.apache.org/r/42854/diff/
> 
> 
> Testing
> -------
> 
> Tested
> 
> 
> Thanks,
> 
> Paul Isaychuk
> 
>


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