----------------------------------------------------------- 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 > >
