----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30280/#review69674 -----------------------------------------------------------
Ship it! Ship It! - Josh Elser On Jan. 26, 2015, 7:04 p.m., Corey Nolet wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30280/ > ----------------------------------------------------------- > > (Updated Jan. 26, 2015, 7:04 p.m.) > > > Review request for accumulo, Sean Busbey, Christopher Tubbs, Eric Newton, > Josh Elser, and kturner. > > > Bugs: ACCUMULO-3533 > https://issues.apache.org/jira/browse/ACCUMULO-3533 > > > Repository: accumulo > > > Description > ------- > > ACCUMULO-3533 Moving the getConfiguration logic which uses reflection to > guarantee Hadoop 1 & 2 compatiblity to its own util class outside of the > public API. Making the AbstractInputFormat.getConfiguration() protected once > again. > > > Diffs > ----- > > > core/src/main/java/org/apache/accumulo/core/client/mapreduce/AbstractInputFormat.java > bcbfddc > > core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormat.java > c68dd56 > > core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloMultiTableInputFormat.java > 010a94f > > core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java > 0f495f0 > > core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java > a60cb80 > core/src/main/java/org/apache/accumulo/core/util/HadoopCompatUtil.java > PRE-CREATION > > examples/simple/src/main/java/org/apache/accumulo/examples/simple/mapreduce/TeraSortIngest.java > 1b8cbaf > > Diff: https://reviews.apache.org/r/30280/diff/ > > > Testing > ------- > > Basic build with unit tests. > > > Thanks, > > Corey Nolet > >