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

Review request for accumulo.


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


Repository: accumulo


Description
-------

These changes for 1.6 for ACCUMULO-2520 are very different than the changes 
made in 1.4 and 1.5.  I merged the changes from 1.5 to 1.6 using -sours (as 
opposed to resolving conflicts in a merge in a radically different way).  This 
patch makes the following changes.

 * adds test for bad delete markers
 * made changes do disallow a delete marker of hdfs://nn/, it used to pass even 
though "" and "/" did not in 1.6
 * logs a warning about invalid delete markers instead of derailing garbage 
collection


Diffs
-----

  
server/gc/src/main/java/org/apache/accumulo/gc/GarbageCollectionAlgorithm.java 
464d0d9 
  server/gc/src/test/java/org/apache/accumulo/gc/GarbageCollectionTest.java 
1e9c1dd 
  
test/src/test/java/org/apache/accumulo/test/functional/GarbageCollectorIT.java 
3a9b404 

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


Testing
-------

ran added/changed test


Thanks,

kturner

Reply via email to