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

(Updated Jan. 22, 2014, 8:25 p.m.)


Review request for accumulo and Eric Newton.


Changes
-------

Noticed an error in one of the log messages; we don't retry closing when 
there's a failure.


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


Repository: accumulo


Description
-------

ACCUMULO-2213 Make writer recovery in the Tracer more robust.
    
    * Cleans up writer reseting in the TraceServer, avoids overly broad 
catching.
    * tones down log levels in TraceServer to WARN because trace information is 
transient and we retry everything.


Diffs (updated)
-----

  src/server/src/main/java/org/apache/accumulo/server/trace/TraceServer.java 
4d89e9c13f89e71e674875db32cf267e045d6131 

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


Testing
-------

running on cluster with injected failures now


Thanks,

Sean Busbey

Reply via email to