> On Oct. 28, 2015, 10:17 p.m., Venkat Ranganathan wrote: > > common/src/main/java/org/apache/falcon/entity/FeedHelper.java, line 813 > > <https://reviews.apache.org/r/39711/diff/4/?file=1111629#file1111629line813> > > > > Shouldn't the default be the same before? Wouldn't this be change in > > behavior.
This doesn't affect the old retention. This behaviour is applicable only to retention in lifecycle. Even in lifecycle we try to fall back to old behaviour as much as possible (as seen from the diff on left side) but it causes issues as mentioned in this JIRA and hence the need for this change. - Ajay ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39711/#review104358 ----------------------------------------------------------- On Oct. 28, 2015, 6:04 p.m., Ajay Yadava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39711/ > ----------------------------------------------------------- > > (Updated Oct. 28, 2015, 6:04 p.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1560 > https://issues.apache.org/jira/browse/FALCON-1560 > > > Repository: falcon-git > > > Description > ------- > > Lifecycle does not allow feed with frequency greater than days(1) > > > Diffs > ----- > > common/src/main/java/org/apache/falcon/entity/FeedHelper.java 5c252a8 > common/src/test/java/org/apache/falcon/entity/FeedHelperTest.java 4020d36 > > common/src/test/java/org/apache/falcon/entity/parser/FeedEntityParserTest.java > 905be68 > > Diff: https://reviews.apache.org/r/39711/diff/ > > > Testing > ------- > > Added unit test for the scenarios. > > > Thanks, > > Ajay Yadava > >
