----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20467/#review40947 -----------------------------------------------------------
Ship it! test/src/test/java/org/apache/accumulo/test/functional/RecoveryWithEmptyRFileIT.java <https://reviews.apache.org/r/20467/#comment74266> speeling - Josh Elser On April 17, 2014, 9:45 p.m., Sean Busbey wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20467/ > ----------------------------------------------------------- > > (Updated April 17, 2014, 9:45 p.m.) > > > Review request for accumulo and kturner. > > > Bugs: ACCUMULO-2654 > https://issues.apache.org/jira/browse/ACCUMULO-2654 > > > Repository: accumulo > > > Description > ------- > > Results of merging 1.5.2-SNAPSHOT version forward to 1.6.0-SNAPSHOT (involved > some package / class changes for information about e.g. compression algos) > and then updating the functional test to be an IT. > > Main review here is of the IT version of hte test. > > > Diffs > ----- > > core/src/main/java/org/apache/accumulo/core/file/rfile/CreateEmpty.java > PRE-CREATION > core/src/main/java/org/apache/accumulo/core/file/rfile/RFileOperations.java > b361413 > > core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Compression.java > 66ca07f > test/src/test/java/org/apache/accumulo/test/functional/ReadWriteIT.java > cb803f8 > > test/src/test/java/org/apache/accumulo/test/functional/RecoveryWithEmptyRFileIT.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/20467/diff/ > > > Testing > ------- > > ran through IT again. tested creating files on real cluster. > > > Thanks, > > Sean Busbey > >