accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ke...@deenlo.com
Subject Review Request 19601: ACCUMULO-2520 1.6.0-SNAPSHOT patch
Date Tue, 25 Mar 2014 02:53:51 GMT

-----------------------------------------------------------
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


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