accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sean Busbey" <s...@manvsbeard.com>
Subject Re: Review Request 17132: ACCUMULO-2224 Make ZooSession more resiliant in the face of transient DNS issues.
Date Tue, 21 Jan 2014 18:12:56 GMT

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

(Updated Jan. 21, 2014, 6:12 p.m.)


Review request for accumulo.


Changes
-------

updated to break the ttl lookup into a utility method (since I think I need it in ACCUMULO-2225
as well), added a test.


Bugs: ACCUMULO-2224
    https://issues.apache.org/jira/browse/ACCUMULO-2224


Repository: accumulo


Description (updated)
-------

* retries if host is not found, up to 2xZK timeout (same as other IOExceptions), rather than
bailing on any host name problem.
* adds utility method for getting the max time the JVM will cache host failures
* add test for said method


Diffs (updated)
-----

  src/core/src/main/java/org/apache/accumulo/core/util/AddressUtil.java 0b821289d2ea6739e664b33c5db3d5050306f7fe

  src/core/src/main/java/org/apache/accumulo/core/zookeeper/ZooSession.java e64f0c52e51b9339930f6a70564fcc48031df687

  src/core/src/test/java/org/apache/accumulo/core/util/AddressUtilTest.java f46f4277d8c12693e1e32cfa6870a88f96742753

  src/core/src/test/resources/log4j.properties PRE-CREATION 

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


Testing
-------

functional + ci in progress


Thanks,

Sean Busbey


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