----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25095/#review51699 -----------------------------------------------------------
falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismFeedUpdateTest.java <https://reviews.apache.org/r/25095/#comment90245> As you are making changes to the test - can you please add javadoc for it ? - Raghav Gautam On Aug. 27, 2014, 4:50 a.m., Paul Isaychuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25095/ > ----------------------------------------------------------- > > (Updated Aug. 27, 2014, 4:50 a.m.) > > > Review request for Falcon. > > > Bugs: 643 > https://issues.apache.org/jira/browse/643 > > > Repository: falcon-git > > > Description > ------- > > Test cases didn't match their scenarios. NoOutputProcessTest amended, > PrismFeedUpdateTest partially complete. > > > Diffs > ----- > > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/NoOutputProcessTest.java > df63483 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismFeedUpdateTest.java > b6bf6d6 > > Diff: https://reviews.apache.org/r/25095/diff/ > > > Testing > ------- > > Tested. PismFeedUpdateTest passes. NoOutputProcessTest fails due to another > issue. > > > Thanks, > > Paul Isaychuk > >
