----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19601/#review38491 -----------------------------------------------------------
Ship it! Ship It! - Sean Busbey On March 25, 2014, 2:53 a.m., kturner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19601/ > ----------------------------------------------------------- > > (Updated March 25, 2014, 2:53 a.m.) > > > 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 > >
