> On Oct. 27, 2015, 5:23 a.m., pavan kumar kolamuri wrote: > > unit/src/test/java/org/apache/falcon/unit/TestFalconUnit.java, line 339 > > <https://reviews.apache.org/r/39657/diff/1/?file=1107830#file1107830line339> > > > > There is already test case for Feed Retention right is this needed > > again ?
It is required to test the feedListing api - Narayan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39657/#review104137 ----------------------------------------------------------- On Oct. 26, 2015, 4:01 p.m., Narayan Periwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39657/ > ----------------------------------------------------------- > > (Updated Oct. 26, 2015, 4:01 p.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 > >
