----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19601/#review38426 -----------------------------------------------------------
server/gc/src/main/java/org/apache/accumulo/gc/GarbageCollectionAlgorithm.java <https://reviews.apache.org/r/19601/#comment70590> Does this exception require immediate operator attention? I think no, right? At worst, we're leaking files in HDFS. but probably not so fast that they need to do something? If it doesn't require immediate attention, I'd like to lower this to WARN. - 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 > >
