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 17157: Add a 3rd argument to functional test RunTests.java to allow a timeout factor for mapred.task.timeout and run.py -f flag
Date Sun, 09 Feb 2014 23:23:53 GMT

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

(Updated Feb. 9, 2014, 11:23 p.m.)


Review request for accumulo.


Changes
-------

ACCUMULO-2005.v7.patch.txt includes following:

1)  Annotation Parameter strTimeoutFactor was changed from String to Integer, intTimeoutFactor
with a default value of 1.
2)  static private Integer timeoutFactor was eliminated and opts.intTimeoutFactor was used
to assigned directly to static final String TIMEOUT_FACTOR.
3)  NumberFormatException was no longer needed.


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


Diffs
-----

  test/src/main/java/org/apache/accumulo/test/functional/RunTests.java 91ee66e 
  test/system/auto/README 7d18e8b 

Diff: https://reviews.apache.org/r/17157/diff/


Testing
-------

Testing was done on small 5-nodes cluster to ensure MR job complete successfully instead of
failing out due to timeouts that lead to killed attempts.


File Attachments (updated)
----------------

ACCUMULO-2005.v7.patch.txt
  https://reviews.apache.org/media/uploaded/files/2014/02/09/4eceb3be-f5a4-487f-84c4-01d3833f5507__ACCUMULO-2005.v7.patch.txt


Thanks,

Hung Pham


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