----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38794/#review100796 -----------------------------------------------------------
prism/src/main/java/org/apache/falcon/resource/AbstractSchedulableEntityManager.java (line 159) <https://reviews.apache.org/r/38794/#comment158042> Can we use StringUtils.isEmpty. - Peeyush Bishnoi On Sept. 27, 2015, 11:25 p.m., Ajay Yadava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38794/ > ----------------------------------------------------------- > > (Updated Sept. 27, 2015, 11:25 p.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1473 > https://issues.apache.org/jira/browse/FALCON-1473 > > > Repository: falcon-git > > > Description > ------- > > This api lists all the pending feed instances in a given time range which > have missed sla. > > > Diffs > ----- > > client/src/main/java/org/apache/falcon/ResponseHelper.java a13682b > client/src/main/java/org/apache/falcon/cli/FalconCLI.java c914649 > client/src/main/java/org/apache/falcon/client/FalconClient.java 981559b > client/src/main/java/org/apache/falcon/entity/v0/SchemaHelper.java 62b810c > > client/src/main/java/org/apache/falcon/resource/SchedulableEntityInstance.java > f5be63d > > client/src/main/java/org/apache/falcon/resource/SchedulableEntityInstanceResult.java > PRE-CREATION > common/src/main/java/org/apache/falcon/entity/FeedHelper.java 572923b > common/src/main/java/org/apache/falcon/entity/parser/FeedEntityParser.java > 4f5599e > common/src/main/resources/startup.properties 9db460c > docs/src/site/twiki/FalconCLI.twiki 4f72bf8 > docs/src/site/twiki/restapi/FeedSLA.twiki PRE-CREATION > docs/src/site/twiki/restapi/ResourceList.twiki ea3e3b6 > prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java > 2682257 > > prism/src/main/java/org/apache/falcon/resource/AbstractSchedulableEntityManager.java > 3280789 > > prism/src/main/java/org/apache/falcon/resource/proxy/SchedulableEntityManagerProxy.java > 47038e5 > prism/src/main/java/org/apache/falcon/service/FeedSLAMonitoringService.java > 8bf43b8 > prism/src/test/java/org/apache/falcon/service/FeedSLAMonitoringTest.java > PRE-CREATION > src/conf/startup.properties 8f3bc35 > > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java > 1c0fc74 > > Diff: https://reviews.apache.org/r/38794/diff/ > > > Testing > ------- > > Unit tests added. > Tested manually by deploying. > > > Thanks, > > Ajay Yadava > >
