----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39657/#review104189 -----------------------------------------------------------
prism/src/main/java/org/apache/falcon/resource/AbstractInstanceManager.java (line 562) <https://reviews.apache.org/r/39657/#comment162477> getProperties is not throwing any exception now. Can you please check ? prism/src/main/java/org/apache/falcon/resource/AbstractInstanceManager.java (line 593) <https://reviews.apache.org/r/39657/#comment162478> same here catch not required prism/src/main/java/org/apache/falcon/resource/AbstractInstanceManager.java (line 624) <https://reviews.apache.org/r/39657/#comment162479> here as well - pavan kumar kolamuri On Oct. 27, 2015, 6:24 a.m., Narayan Periwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39657/ > ----------------------------------------------------------- > > (Updated Oct. 27, 2015, 6:24 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1528 > https://issues.apache.org/jira/browse/FALCON-1528 > > > Repository: falcon-git > > > Description > ------- > > Instance Management Api's should be supported in Falcon Unit. > > > Diffs > ----- > > client/src/main/java/org/apache/falcon/client/AbstractFalconClient.java > 91d5324 > client/src/main/java/org/apache/falcon/client/FalconClient.java 27510f6 > oozie/src/main/java/org/apache/oozie/client/LocalOozieClientCoordProxy.java > PRE-CREATION > oozie/src/main/java/org/apache/oozie/client/LocalProxyOozieClient.java > c2100d1 > prism/src/main/java/org/apache/falcon/resource/AbstractInstanceManager.java > 606f741 > unit/src/main/java/org/apache/falcon/unit/FalconUnitClient.java b5afae3 > unit/src/main/java/org/apache/falcon/unit/LocalInstanceManager.java 1503b28 > unit/src/test/java/org/apache/falcon/unit/FalconUnitTestBase.java ac478f4 > unit/src/test/java/org/apache/falcon/unit/TestFalconUnit.java 8cdbd88 > unit/src/test/java/org/apache/falcon/unit/examples/JavaSleepExample.java > PRE-CREATION > unit/src/test/resources/sleepWorkflow.xml PRE-CREATION > > Diff: https://reviews.apache.org/r/39657/diff/ > > > Testing > ------- > > Done. > > > Thanks, > > Narayan Periwal > >
