----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41742/#review112203 -----------------------------------------------------------
falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceStatusTest.java (line 343) <https://reviews.apache.org/r/41742/#comment172544> Is sleep required here, I mean we have already waited for instance to reach suspended state, so why is another wait required? falcon-regression/merlin/src/test/java/org/apache/falcon/regression/lineage/ListFeedInstancesTest.java (line 136) <https://reviews.apache.org/r/41742/#comment173031> Isn't it 2nd & 3rd instances which are getting killed? - Ajay Yadava On Dec. 28, 2015, 4:05 p.m., Paul Isaychuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41742/ > ----------------------------------------------------------- > > (Updated Dec. 28, 2015, 4:05 p.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1697 > https://issues.apache.org/jira/browse/FALCON-1697 > > > Repository: falcon-git > > > Description > ------- > > Fixes related to stabilization of tests which are based on instance lifecycle > by adding timeouts, separating data required for different instances etc, > optimization scenario. > > > Diffs > ----- > > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/InstanceUtil.java > 3d05ae9 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceStatusTest.java > 6493133 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java > f673314 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/lineage/ListFeedInstancesTest.java > 7ad4c8e > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/lineage/ListProcessInstancesTest.java > 43bdd87 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/searchUI/EntityPageTest.java > 4ad775e > > Diff: https://reviews.apache.org/r/41742/diff/ > > > Testing > ------- > > Testing done. > > > Thanks, > > Paul Isaychuk > >
