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

(Updated Jan. 29, 2015, 10:47 p.m.)


Review request for accumulo and Eric Newton.


Changes
-------

Changed code to do nothing w/ span Trace.on fails


Bugs: ACCUMULO-3462 and ACCUMULO-3541
    https://issues.apache.org/jira/browse/ACCUMULO-3462
    https://issues.apache.org/jira/browse/ACCUMULO-3541


Repository: accumulo


Description
-------

Fixes compaction logging bug and bug with compaction state not being reset.  
Patch builds on work already committed for ACCUMULO-3462.

I am slightly nervous about the changes w/ control flow, but I feel this is the 
best solution to avoid duplicate logging.


Diffs (updated)
-----

  server/tserver/src/main/java/org/apache/accumulo/tserver/Tablet.java 7420ec4 

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


Testing
-------

Still running ITs.  Getting this patch out for review so we can move forward 
for 1.6.2.


Thanks,

kturner

Reply via email to