falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "samarth gupta" <samarthgupta...@gmail.com>
Subject Re: Review Request 31353: [Falcon-1046] Add test for process update with user feature
Date Thu, 26 Feb 2015 05:04:12 GMT

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



falcon-regression/merlin/src/test/java/org/apache/falcon/regression/AuthorizationTest.java
<https://reviews.apache.org/r/31353/#comment120730>

    how was the validity set till now ? why do we need to set it explicitly now ?



falcon-regression/merlin/src/test/java/org/apache/falcon/regression/AuthorizationTest.java
<https://reviews.apache.org/r/31353/#comment120731>

    all these test seems to be disabled. If they are passing now can we enable them ?


- samarth gupta


On Feb. 24, 2015, 1:12 p.m., Karishma Gulati wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31353/
> -----------------------------------------------------------
> 
> (Updated Feb. 24, 2015, 1:12 p.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> More details can be found at : https://issues.apache.org/jira/browse/FALCON-430.
> Example Test Case:
> FeedA Scheduled by U1, Dependent Process by U2, FeedA Updated by U1 -> New Bundle
for FeedA with U1, New Bundle for Process with U2 (and not U1) created.
> 
> Similarly, have identified a couple of other test cases.
> 
> 
> Diffs
> -----
> 
>   falcon-regression/merlin/src/test/java/org/apache/falcon/regression/AuthorizationTest.java
4787ef1 
> 
> Diff: https://reviews.apache.org/r/31353/diff/
> 
> 
> Testing
> -------
> 
> Added Tests to Authorization Test.
> 
> 
> Thanks,
> 
> Karishma Gulati
> 
>


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