----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24980/#review51312 -----------------------------------------------------------
falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceRerunTest.java <https://reviews.apache.org/r/24980/#comment89493> process should probably be made a method local variable, becasue it is being reassigned in every method. - Raghav Gautam On Aug. 22, 2014, 7:32 a.m., Paul Isaychuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24980/ > ----------------------------------------------------------- > > (Updated Aug. 22, 2014, 7:32 a.m.) > > > Review request for Falcon. > > > Repository: falcon-git > > > Description > ------- > > Makes code more attractive > > > Diffs > ----- > > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/EmbeddedPigScriptTest.java > 1973bf8 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/FeedInstanceStatusTest.java > ff227d6 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceKillsTest.java > e7b2616 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceRerunTest.java > 119d871 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessLibPathTest.java > fc8e4a8 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/RetentionTest.java > 85bd770 > > Diff: https://reviews.apache.org/r/24980/diff/ > > > Testing > ------- > > tested > > > Thanks, > > Paul Isaychuk > >
