----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41400/#review111750 -----------------------------------------------------------
falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/FeedSLATest.java (line 85) <https://reviews.apache.org/r/41400/#comment172004> Clear feedClusters in before Method since you're doing it in all tests. falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/FeedSLATest.java (line 108) <https://reviews.apache.org/r/41400/#comment172005> move to before Method falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/FeedSLATest.java (line 138) <https://reviews.apache.org/r/41400/#comment172006> move to before Method falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/FeedSLATest.java (line 164) <https://reviews.apache.org/r/41400/#comment172007> Move to before Method falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/FeedSLATest.java (line 192) <https://reviews.apache.org/r/41400/#comment172008> move to before Method falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/ProcessSLATest.java (line 86) <https://reviews.apache.org/r/41400/#comment171998> Move this line to before method. And declare processMerlin a global variable. falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/ProcessSLATest.java (line 101) <https://reviews.apache.org/r/41400/#comment171999> Move to before Method falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/ProcessSLATest.java (line 116) <https://reviews.apache.org/r/41400/#comment172001> Move to before Method. falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/ProcessSLATest.java (line 137) <https://reviews.apache.org/r/41400/#comment172002> Move to before Method falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/ProcessSLATest.java (line 153) <https://reviews.apache.org/r/41400/#comment172003> Move to before Method - Karishma Gulati On Dec. 15, 2015, 2:48 p.m., PRAGYA MITTAL wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41400/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2015, 2:48 p.m.) > > > Review request for Falcon and Ajay Yadava. > > > Bugs: FALCON-1566 > https://issues.apache.org/jira/browse/FALCON-1566 > > > Repository: falcon-git > > > Description > ------- > > Add test for SLA monitoring API > > > Diffs > ----- > > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/helpers/entity/AbstractEntityHelper.java > e406cae > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/response/ServiceResponse.java > 55e862c > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/Util.java > ccd083b > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/FeedSLATest.java > 28ddbd7 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessSLATest.java > f4c9b30 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/FeedSLAMonitoring.java > PRE-CREATION > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/FeedSLATest.java > PRE-CREATION > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/SLA/ProcessSLATest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/41400/diff/ > > > Testing > ------- > > Tetsed in distributed mode. > > > Thanks, > > PRAGYA MITTAL > >
