> On Sept. 17, 2015, 12:20 p.m., pavan kumar kolamuri wrote: > > common/src/main/java/org/apache/falcon/entity/parser/ProcessEntityParser.java, > > line 85 > > <https://reviews.apache.org/r/38459/diff/1/?file=1075909#file1075909line85> > > > > Before setting process end time shouldn't we check feed end times of > > this process ? Feed end time might be less than this right ? > > Balu Vellanki wrote: > There are two ways to handle the scenario where a process is being > submitted with it's feed having an endtime. > 1) Get feed's endtime and use it for process. > or > 2) Set the end to DateUtil.OPTIONAL_DATE.NEVER.getDate() and let process > validation throw an exception. > > I think that Falcon should not make assumptions about user intent in > option (1), so prefer option 2. > > pavan kumar kolamuri wrote: > I agree balu, but there is chance that it will always throw exception, if > the feed end time is set and this feature will never come into useful.
As of now validity check is applied for the first instance of the process.So, IMO falcon would never check for feed end date to be less than process end date. I have raised a JIRA for the same. https://issues.apache.org/jira/browse/FALCON-1400. Can you please site an example where the check would fail? - PRAGYA ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38459/#review99373 ----------------------------------------------------------- On Sept. 17, 2015, 11:01 a.m., PRAGYA MITTAL wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38459/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2015, 11:01 a.m.) > > > Review request for Falcon and Ajay Yadava. > > > Bugs: FALCON-1417 > https://issues.apache.org/jira/browse/FALCON-1417 > > > Repository: falcon-git > > > Description > ------- > > Added optional date functionality. > > > Diffs > ----- > > > common/src/main/java/org/apache/falcon/entity/parser/CrossEntityValidations.java > 2696552 > common/src/main/java/org/apache/falcon/entity/parser/FeedEntityParser.java > 992fc51 > > common/src/main/java/org/apache/falcon/entity/parser/ProcessEntityParser.java > 48a4286 > common/src/main/java/org/apache/falcon/util/DateUtil.java e736340 > > common/src/test/java/org/apache/falcon/entity/parser/FeedEntityParserTest.java > d203b7c > > common/src/test/java/org/apache/falcon/entity/parser/ProcessEntityParserTest.java > 77f6a77 > > Diff: https://reviews.apache.org/r/38459/diff/ > > > Testing > ------- > > Added test cases for the same. > > > Thanks, > > PRAGYA MITTAL > >
