----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16908/#review32124 -----------------------------------------------------------
src/server/src/main/java/org/apache/accumulo/server/test/functional/RunTests.java <https://reviews.apache.org/r/16908/#comment60807> non-constant variables should be named in mixedLowerCase (sorry I missed this in the initial review) src/server/src/main/java/org/apache/accumulo/server/test/functional/RunTests.java <https://reviews.apache.org/r/16908/#comment60805> non-constant variables should be named in mixedLowerCase (sorry I missed this in the initial review) - Sean Busbey On Jan. 16, 2014, 1:17 p.m., Hung Pham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16908/ > ----------------------------------------------------------- > > (Updated Jan. 16, 2014, 1:17 p.m.) > > > Review request for accumulo. > > > Bugs: ACCUMULO-2005 > https://issues.apache.org/jira/browse/ACCUMULO-2005 > > > Repository: accumulo > > > Description > ------- > > ACCUMULO-2005: Add a 3rd argument to functional test RunTests.java to allow a > timeout factor for mapred.task.timeout and run.py -f flag. Also changed 'ex' > to 'e' on line 274 of > src/server/src/main/java/org/apache/accumulo/server/monitor/servlets/DefaultServlet.java > > > Diffs > ----- > > > src/server/src/main/java/org/apache/accumulo/server/test/functional/RunTests.java > 1a19bb5 > test/system/auto/README 4773393 > > Diff: https://reviews.apache.org/r/16908/diff/ > > > Testing > ------- > > Testing was done on small cluster (4 nodes: master + 3 workers) to ensure MR > job complete successfully instead of failing out due to timeouts that lead to > killed attempts. Test results can be provided upon request. > > > Thanks, > > Hung Pham > >
