accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ke...@deenlo.com
Subject Review Request 19546: ACCUMULO-2520 made GC validate data read from !METADATA table
Date Fri, 21 Mar 2014 19:27:04 GMT

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

Review request for accumulo.


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


Repository: accumulo


Description
-------

This change should make the GC more robust against the case of it deleting things it should
not.  A concern with this change is that may make the GC fail in a situation where it should
not, which could cause hdfs to fill.


Diffs
-----

  src/server/src/main/java/org/apache/accumulo/server/gc/SimpleGarbageCollector.java 22c3c0e

  src/server/src/test/java/org/apache/accumulo/server/gc/SimpleGarbageCollectorTest.java PRE-CREATION

  src/server/src/test/java/org/apache/accumulo/server/gc/TestConfirmDeletes.java be444dd 

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


Testing
-------

Added unit test
Ran 1.4.5-SNAPSHOT and verified GC worked and deleted an orphaned file


Thanks,

kturner


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