> On March 5, 2015, 8:22 p.m., Raghav Gautam wrote: > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/OozieUtil.java, > > line 530 > > <https://reviews.apache.org/r/31726/diff/1/?file=884395#file884395line530> > > > > Wouldn't this return missing dependency of only the last instance ? > > > > Also, consider implementinng getMissingDependenciesForBundle() method. > > createMissingDependenciesForBundle() can use this new method.
Thanks for pointing this out. - Karishma ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31726/#review75375 ----------------------------------------------------------- On March 5, 2015, 6:57 a.m., Karishma Gulati wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31726/ > ----------------------------------------------------------- > > (Updated March 5, 2015, 6:57 a.m.) > > > Review request for Falcon. > > > Repository: falcon-git > > > Description > ------- > > Falcon processes or feed replication provides status on instances. When the > instance status is WAITING for input, users have difficulty in identifying > what feed is missing. In general it would very helpful to users to get feed > availability status natively through falcon. > > More description given at : https://issues.apache.org/jira/browse/FALCON-761 > > > Diffs > ----- > > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/helpers/entity/AbstractEntityHelper.java > 7d38261 > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/InstanceUtil.java > 5d4e657 > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/OozieUtil.java > 3cc171f > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/Util.java > ab09ff1 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/FeedInstanceListingTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/31726/diff/ > > > Testing > ------- > > > Thanks, > > Karishma Gulati > >
