----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39632/#review113478 -----------------------------------------------------------
scheduler/src/main/java/org/apache/falcon/notification/service/impl/DataAvailabilityService.java (line 115) <https://reviews.apache.org/r/39632/#comment174213> I think Data Service shouldn't care which location type it is and this should be removed. scheduler/src/main/java/org/apache/falcon/notification/service/impl/DataAvailabilityService.java (line 216) <https://reviews.apache.org/r/39632/#comment174216> I believe this is a typo and should be a comma instead of "+" scheduler/src/main/java/org/apache/falcon/notification/service/impl/DataAvailabilityService.java (line 228) <https://reviews.apache.org/r/39632/#comment174219> IMHO we don't have a usecase for handlers currently and we should avoid adding it until we have one. scheduler/src/main/java/org/apache/falcon/predicate/Predicate.java (line 169) <https://reviews.apache.org/r/39632/#comment174211> We shouldn't accept null paths & locations. - Ajay Yadava On Jan. 8, 2016, 12:30 p.m., pavan kumar kolamuri wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39632/ > ----------------------------------------------------------- > > (Updated Jan. 8, 2016, 12:30 p.m.) > > > Review request for Falcon. > > > Bugs: https://issues.apache.org/jira/browse/FALCON-1230 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/FALCON-1230 > > > Repository: falcon-git > > > Description > ------- > > Data Availability Service for Falcon . It will notify When Data paths are > available. > > > Diffs > ----- > > > scheduler/src/main/java/org/apache/falcon/execution/ProcessExecutionInstance.java > 72e5558 > > scheduler/src/main/java/org/apache/falcon/notification/service/event/DataEvent.java > 1036339 > > scheduler/src/main/java/org/apache/falcon/notification/service/impl/DataAvailabilityService.java > 7ffb351 > > scheduler/src/main/java/org/apache/falcon/notification/service/request/DataNotificationRequest.java > 8393de0 > scheduler/src/main/java/org/apache/falcon/predicate/Predicate.java c7b4f12 > > scheduler/src/test/java/org/apache/falcon/execution/FalconExecutionServiceTest.java > d66972c > > scheduler/src/test/java/org/apache/falcon/notification/service/DataAvailabilityServiceTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/39632/diff/ > > > Testing > ------- > > Unit Tests are written > > > Thanks, > > pavan kumar kolamuri > >
