accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sean Busbey" <s...@manvsbeard.com>
Subject Re: Review Request 19716: ACCUMULO-2564 Backport changes to unify Hadoop 1/2
Date Fri, 28 Mar 2014 20:46:51 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19716/#review38943
-----------------------------------------------------------

Ship it!


Ship It!

- Sean Busbey


On March 28, 2014, 8:34 p.m., Mike Drob wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19716/
> -----------------------------------------------------------
> 
> (Updated March 28, 2014, 8:34 p.m.)
> 
> 
> Review request for accumulo, Adam Fuchs, Sean Busbey, Eric Newton, and Josh Elser.
> 
> 
> Bugs: ACCUMULO-2564
>     https://issues.apache.org/jira/browse/ACCUMULO-2564
> 
> 
> Repository: accumulo
> 
> 
> Description
> -------
> 
> ACCUMULO-2564 Replace more hadoop 1/2 incompat
> 
> Found more instances of context.getConfiguration that need to be reflected
> 
> ACCUMULO-2564 Swap out AIF to use reflection
> 
> Use reflections to get the configuration from the context for hadoop 1/2
> 
> ACCUMULO-2564 Backport changes from ACCUMULO-1809
> 
> Author: Eric Newton
> Reason: use reflection tricks to update counters
> 
> ACCUMULO-2564 Backport changes to unify Hadoop 1/2
> 
> This is a backport of the changes originally made for 1.5.0
> under ACCUMULO-1421 for binary compatability between Hadoop versions
> 1 and 2 with the same Accumulo artifacts. This commit is based on the
> following original work:
> 
> * c9c0d45 (Adam Fuchs)
> * d7ba6ca (Christopher Tubbs)
> * 261cf36 (Eric Newton)
> * cc3c2d8 (Eric Newton)
> 
> 
> Diffs
> -----
> 
>   src/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormat.java
7cfab8b6e8f2199620c36a50ac067dee53aab6a9 
>   src/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloInputFormat.java
c9a70eb24ffb21d845e0915d0dc7cebe8035a37e 
>   src/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java
ed0aebf7476d8db6a968e858c9c4c892dac78bc5 
>   src/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
a11096c3f855104db4e3b78a49f3fb039ac268ec 
>   src/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/partition/RangePartitioner.java
ae537f9cd919a3821e40cc0a99bf31815b5bb235 
>   src/core/src/main/java/org/apache/accumulo/core/util/ContextFactory.java 5a1c2ef6184aeb56408b3cab451bc2477377264f

>   src/server/src/main/java/org/apache/accumulo/server/Accumulo.java 184692c48ce8013067053c1d0f0dd6a7a889473a

>   src/server/src/main/java/org/apache/accumulo/server/master/LogSort.java 1c384a3779179927d56546138d2cb6d4023c941b

>   src/server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousMoru.java
1c384ccfd323eec761172c86ebbf970feadf0f9e 
>   src/server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousVerify.java
9441cf5314383705260d6fa85e69597525de421c 
> 
> Diff: https://reviews.apache.org/r/19716/diff/
> 
> 
> Testing
> -------
> 
> Functional tests pass when running a 1.4.5 default profile build against cdh4.5.0 (hadoop
2).
> 
> 
> Thanks,
> 
> Mike Drob
> 
>


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