----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25803/#review53896 -----------------------------------------------------------
falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/Config.java <https://reviews.apache.org/r/25803/#comment93762> What is the reason for adding this change? Add comments here as to why we are doing it. - Arpit Gupta On Sept. 18, 2014, 11:16 p.m., Raghav Gautam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25803/ > ----------------------------------------------------------- > > (Updated Sept. 18, 2014, 11:16 p.m.) > > > Review request for Falcon. > > > Bugs: falcon-745 > https://issues.apache.org/jira/browse/falcon-745 > > > Repository: falcon-git > > > Description > ------- > > LogUtil pulls a lot of logs from oozie this can add burden to already long > running tests. > It would benefit from having finer control over what to pull > It will also benefit if these controls can be exercised while the job is > running > > > Diffs > ----- > > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/Config.java > 9ae3f0b > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/LogUtil.java > 42583fa > > Diff: https://reviews.apache.org/r/25803/diff/ > > > Testing > ------- > > both of the added options were tested > > > Thanks, > > Raghav Gautam > >
