accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hung Pham" <hp...@clouderagovt.com>
Subject Re: Review Request 16908: Add a 3rd argument to functional test RunTests.java to allow a timeout factor for mapred.task.timeout and run.py -f flag
Date Fri, 17 Jan 2014 17:22:20 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16908/
-----------------------------------------------------------

(Updated Jan. 17, 2014, 5:22 p.m.)


Review request for accumulo.


Changes
-------

Following changes are in this version 4 patch:
1)  Non-constant variable timeout_factor was changed to timeoutFactor.
2)  non-constant variable mapper_timeout_factor was changed to mapperTimeoutFactor.


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 (updated)
-----

  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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message