----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26912/#review60508 -----------------------------------------------------------
common/src/main/java/org/apache/falcon/entity/parser/ProcessEntityParser.java <https://reviews.apache.org/r/26912/#comment101899> Variable names are still slaStart/slaEnd. Looks like slaStart is validated only if slaEnd is populated. Are we proposing that slaStart has no significance unless slaEnd is not specified - Srikanth Sundarrajan On Nov. 6, 2014, 9:31 a.m., Ajay Yadava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26912/ > ----------------------------------------------------------- > > (Updated Nov. 6, 2014, 9:31 a.m.) > > > Review request for Falcon and Srikanth Sundarrajan. > > > Bugs: https://issues.apache.org/jira/browse/FALCON-722 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/FALCON-722 > > > Repository: falcon-git > > > Description > ------- > > Add SLA for process. > > > Diffs > ----- > > client/src/main/resources/process-0.1.xsd 06a2fe4 > > common/src/main/java/org/apache/falcon/entity/parser/ProcessEntityParser.java > 9be4e85 > > common/src/test/java/org/apache/falcon/entity/parser/ProcessEntityParserTest.java > 80a9cc7 > common/src/test/resources/config/process/process-0.1.xml 99a0376 > docs/src/site/twiki/EntitySpecification.twiki b0dfb7f > > Diff: https://reviews.apache.org/r/26912/diff/ > > > Testing > ------- > > Yes. All tests pass. > > > Thanks, > > Ajay Yadava > >
