Uploaded image for project: 'Accumulo'
  1. Accumulo
  2. ACCUMULO-1421

use reflection to improve compatibility between Hadoop 1 and 2

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.5.0
    • Component/s: client
    • Labels:
      None

      Description

      JobContext switches between an abstract class and an interface in hadoop 1 and 2, causing runtime compatibility problems in the compiled Accumulo bytecode. Avoid the runtime problem by using reflection to access the configuration object stored in the JobContext.

        Issue Links

          Activity

          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.5.2-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.5.2-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.5.2-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.5.2-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.5.2-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.5.2-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.5.2-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.5.2-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.4.5-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.4.5-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.4.5-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.4.5-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.4.5-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.4.5-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.4.5-SNAPSHOT from Mike Drob
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ]

          ACCUMULO-2564 Backport changes to unify Hadoop 1/2

          This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to
          establish binary compatibility with both Hadoop 1 and 2 branches. This
          work was originally done under several commits and issues, as outlined
          below. Additionally, there was some new work that needed to be done,
          but is not applicable to later branches.

          ACCUMULO-1421, commit c9c0d45 by Adam Fuchs
          ACCUMULO-1421, commit d7ba6ca by Christopher Tubbs
          ACCUMULO-1421, commit 261cf36 by Eric Newton
          ACCUMULO-1421, commit cc3c2d8 by Eric Newton

          Issue: ACCUMULO-1809
          Author: Eric Newton
          Reason: use reflection tricks to update counters
          Commit: a7e159219a29ca6f127616fd965aa857900e3f9c

          Issue: n/a
          Reason: Delegate reflection to ContextFactory

          The InsertWithOutputFormat functional test uses ContextFactory, so we
          have to update that to use the reflection hack as well. And while we're
          there, might as well keep the code in one place.

          Show
          jira-bot ASF subversion and git services added a comment - Commit 9e5854fb3ff90c0e80523394401f4103eb4212a2 in accumulo's branch refs/heads/1.4.5-SNAPSHOT from Mike Drob [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9e5854f ] ACCUMULO-2564 Backport changes to unify Hadoop 1/2 This is a backport of the changes made for Accumulo 1.5.0 and 1.5.1 to establish binary compatibility with both Hadoop 1 and 2 branches. This work was originally done under several commits and issues, as outlined below. Additionally, there was some new work that needed to be done, but is not applicable to later branches. ACCUMULO-1421 , commit c9c0d45 by Adam Fuchs ACCUMULO-1421 , commit d7ba6ca by Christopher Tubbs ACCUMULO-1421 , commit 261cf36 by Eric Newton ACCUMULO-1421 , commit cc3c2d8 by Eric Newton Issue: ACCUMULO-1809 Author: Eric Newton Reason: use reflection tricks to update counters Commit: a7e159219a29ca6f127616fd965aa857900e3f9c Issue: n/a Reason: Delegate reflection to ContextFactory The InsertWithOutputFormat functional test uses ContextFactory, so we have to update that to use the reflection hack as well. And while we're there, might as well keep the code in one place.
          Hide
          hudson Hudson added a comment -

          Integrated in Accumulo-Trunk #884 (See https://builds.apache.org/job/Accumulo-Trunk/884/)
          ACCUMULO-1421 use reflection to work around class/interface change in Counter (Revision 1484917)

          Result = SUCCESS
          ecn :
          Files :

          • /accumulo/trunk
          • /accumulo/trunk/assemble
          • /accumulo/trunk/core
          • /accumulo/trunk/examples
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
          • /accumulo/trunk/pom.xml
          • /accumulo/trunk/proxy/README
          • /accumulo/trunk/server
          • /accumulo/trunk/src
          • /accumulo/trunk/test/src/main/java/org/apache/accumulo/test/continuous/ContinuousVerify.java
          Show
          hudson Hudson added a comment - Integrated in Accumulo-Trunk #884 (See https://builds.apache.org/job/Accumulo-Trunk/884/ ) ACCUMULO-1421 use reflection to work around class/interface change in Counter (Revision 1484917) Result = SUCCESS ecn : Files : /accumulo/trunk /accumulo/trunk/assemble /accumulo/trunk/core /accumulo/trunk/examples /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java /accumulo/trunk/pom.xml /accumulo/trunk/proxy/README /accumulo/trunk/server /accumulo/trunk/src /accumulo/trunk/test/src/main/java/org/apache/accumulo/test/continuous/ContinuousVerify.java
          Hide
          hudson Hudson added a comment -

          Integrated in Accumulo-1.5-Hadoop-2.0 #133 (See https://builds.apache.org/job/Accumulo-1.5-Hadoop-2.0/133/)
          ACCUMULO-1421 use reflection to work around class/interface change in Counter (Revision 1484908)

          Result = SUCCESS
          ecn :
          Files :

          • /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/continuous/ContinuousVerify.java
          Show
          hudson Hudson added a comment - Integrated in Accumulo-1.5-Hadoop-2.0 #133 (See https://builds.apache.org/job/Accumulo-1.5-Hadoop-2.0/133/ ) ACCUMULO-1421 use reflection to work around class/interface change in Counter (Revision 1484908) Result = SUCCESS ecn : Files : /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/continuous/ContinuousVerify.java
          Hide
          hudson Hudson added a comment -

          Integrated in Accumulo-Trunk-Hadoop-2.0 #243 (See https://builds.apache.org/job/Accumulo-Trunk-Hadoop-2.0/243/)
          ACCUMULO-1421 use reflection to work around class/interface change in Counter (Revision 1484917)

          Result = SUCCESS
          ecn :
          Files :

          • /accumulo/trunk
          • /accumulo/trunk/assemble
          • /accumulo/trunk/core
          • /accumulo/trunk/examples
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
          • /accumulo/trunk/pom.xml
          • /accumulo/trunk/proxy/README
          • /accumulo/trunk/server
          • /accumulo/trunk/src
          • /accumulo/trunk/test/src/main/java/org/apache/accumulo/test/continuous/ContinuousVerify.java
          Show
          hudson Hudson added a comment - Integrated in Accumulo-Trunk-Hadoop-2.0 #243 (See https://builds.apache.org/job/Accumulo-Trunk-Hadoop-2.0/243/ ) ACCUMULO-1421 use reflection to work around class/interface change in Counter (Revision 1484917) Result = SUCCESS ecn : Files : /accumulo/trunk /accumulo/trunk/assemble /accumulo/trunk/core /accumulo/trunk/examples /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java /accumulo/trunk/pom.xml /accumulo/trunk/proxy/README /accumulo/trunk/server /accumulo/trunk/src /accumulo/trunk/test/src/main/java/org/apache/accumulo/test/continuous/ContinuousVerify.java
          Hide
          hudson Hudson added a comment -

          Integrated in Accumulo-1.5 #131 (See https://builds.apache.org/job/Accumulo-1.5/131/)
          ACCUMULO-1421 use reflection to work around class/interface change in Counter (Revision 1484908)

          Result = SUCCESS
          ecn :
          Files :

          • /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/continuous/ContinuousVerify.java
          Show
          hudson Hudson added a comment - Integrated in Accumulo-1.5 #131 (See https://builds.apache.org/job/Accumulo-1.5/131/ ) ACCUMULO-1421 use reflection to work around class/interface change in Counter (Revision 1484908) Result = SUCCESS ecn : Files : /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/continuous/ContinuousVerify.java
          Hide
          ctubbsii Christopher Tubbs added a comment -

          We will only need to to make one binary package that will
          work against either Hadoop 1 or 2.

          Hopefully that is a true statement.

          Would it be possible or desirable to make Hadoop 1 and Hadoop 2 testing
          modules to test against whichever jars we're not building against?

          Desirable, definitely. Certainly it's possible to run the python test suite on both. It might be possible to create some explicit integration tests in the build, but I'm not entirely sure how the dependencies management would need to work for that (probably would have to be two different sub-modules of the test module).

          Show
          ctubbsii Christopher Tubbs added a comment - We will only need to to make one binary package that will work against either Hadoop 1 or 2. Hopefully that is a true statement. Would it be possible or desirable to make Hadoop 1 and Hadoop 2 testing modules to test against whichever jars we're not building against? Desirable, definitely. Certainly it's possible to run the python test suite on both. It might be possible to create some explicit integration tests in the build, but I'm not entirely sure how the dependencies management would need to work for that (probably would have to be two different sub-modules of the test module).
          Hide
          billie.rinaldi Billie Rinaldi added a comment -

          Let's see if I understand this correctly: we will still support building
          against different versions of Hadoop, but it will not be necessary for
          running Accumulo. We will only need to to make one binary package that will
          work against either Hadoop 1 or 2.

          Would it be possible or desirable to make Hadoop 1 and Hadoop 2 testing
          modules to test against whichever jars we're not building against?

          Show
          billie.rinaldi Billie Rinaldi added a comment - Let's see if I understand this correctly: we will still support building against different versions of Hadoop, but it will not be necessary for running Accumulo. We will only need to to make one binary package that will work against either Hadoop 1 or 2. Would it be possible or desirable to make Hadoop 1 and Hadoop 2 testing modules to test against whichever jars we're not building against?
          Hide
          afuchs Adam Fuchs added a comment -

          May still require a merge to trunk.

          Show
          afuchs Adam Fuchs added a comment - May still require a merge to trunk.
          Hide
          ctubbsii Christopher Tubbs added a comment -

          What version was this fixed for? 1.5? 1.6?

          Show
          ctubbsii Christopher Tubbs added a comment - What version was this fixed for? 1.5? 1.6?
          Hide
          afuchs Adam Fuchs added a comment -

          Eric already fixed it! Guess it was a build management problem on our side.

          Show
          afuchs Adam Fuchs added a comment - Eric already fixed it! Guess it was a build management problem on our side.
          Hide
          ecn Eric Newton added a comment -

          svn update? I fixed that yesterday.

          Show
          ecn Eric Newton added a comment - svn update? I fixed that yesterday.
          Hide
          afuchs Adam Fuchs added a comment -

          Mike Berman found another issue: org.apache.hadoop.net.NetUtils.getInputStream(Ljava/net/Socket;J)Ljava/io/InputStream; became org.apache.hadoop.net.NetUtils.getInputStream(Ljava/net/Socket;J)Lorg/apache/hadoop/net/SocketInputWrapper;. We use this at org.apache.accumulo.core.util.TTimeoutTransport.create(TTimeoutTransport.java:39). This breaks bytecode compatibility, but not source code compatibility. Time for more reflection!

          Show
          afuchs Adam Fuchs added a comment - Mike Berman found another issue: org.apache.hadoop.net.NetUtils.getInputStream(Ljava/net/Socket;J)Ljava/io/InputStream; became org.apache.hadoop.net.NetUtils.getInputStream(Ljava/net/Socket;J)Lorg/apache/hadoop/net/SocketInputWrapper;. We use this at org.apache.accumulo.core.util.TTimeoutTransport.create(TTimeoutTransport.java:39). This breaks bytecode compatibility, but not source code compatibility. Time for more reflection!
          Hide
          busbey Sean Busbey added a comment -

          Adam Fuchs, that's my intention. Having trouble building the 1.5 branch on either OS X or CentOS 6.2, so probably I'm going to be delayed. Different tests failing in both cases.

          Show
          busbey Sean Busbey added a comment - Adam Fuchs , that's my intention. Having trouble building the 1.5 branch on either OS X or CentOS 6.2, so probably I'm going to be delayed. Different tests failing in both cases.
          Hide
          hudson Hudson added a comment -

          Integrated in Accumulo-Trunk #874 (See https://builds.apache.org/job/Accumulo-Trunk/874/)
          merge changes from 1.5

          ACCUMULO-1421 simplify reflection lookup of SafeModeAction
          ACCUMULO-???? changed getConfiguration to be package protected so that it is not in the public API
          ACCUMULO-1274 classpath command shows two java configured classpaths before showing Accumulo configured classpaths.
          ACCUMULO-970 use reasonable defaults for batchwriter in documentation (Revision 1483510)
          ACCUMULO-1421 simplify reflection lookup of SafeModeAction (Revision 1483498)
          ACCUMULO-1421, ACCUMULO-904 merge to trunk (Revision 1483461)
          ACCUMULO-1413,ACCUMULO-1421 merge to trunk (Revision 1483431)

          Result = SUCCESS
          kturner :
          Files :

          • /accumulo/trunk
          • /accumulo/trunk/assemble
          • /accumulo/trunk/core
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          • /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/analytics.tex
          • /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/clients.tex
          • /accumulo/trunk/examples
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
          • /accumulo/trunk/pom.xml
          • /accumulo/trunk/proxy/README
          • /accumulo/trunk/server
          • /accumulo/trunk/src
          • /accumulo/trunk/start/src/main/java/org/apache/accumulo/start/classloader/vfs/AccumuloVFSClassLoader.java
          • /accumulo/trunk/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java
          • /accumulo/trunk/test/src/test/java/org/apache/accumulo/test/ShellServerTest.java

          ecn :
          Files :

          • /accumulo/trunk
          • /accumulo/trunk/assemble
          • /accumulo/trunk/core
          • /accumulo/trunk/examples
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
          • /accumulo/trunk/pom.xml
          • /accumulo/trunk/proxy/README
          • /accumulo/trunk/server
          • /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java
          • /accumulo/trunk/src

          ctubbsii :
          Files :

          • /accumulo/trunk
          • /accumulo/trunk/assemble
          • /accumulo/trunk/core
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/Property.java
          • /accumulo/trunk/examples
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
          • /accumulo/trunk/pom.xml
          • /accumulo/trunk/proxy/README
          • /accumulo/trunk/server
          • /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java
          • /accumulo/trunk/src

          ctubbsii :
          Files :

          • /accumulo/trunk
          • /accumulo/trunk/README
          • /accumulo/trunk/assemble
          • /accumulo/trunk/assemble/pom.xml
          • /accumulo/trunk/assemble/src/main/assemblies/binary-release.xml
          • /accumulo/trunk/core
          • /accumulo/trunk/core/pom.xml
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormat.java
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/TTimeoutTransport.java
          • /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/administration.tex
          • /accumulo/trunk/examples
          • /accumulo/trunk/examples/pom.xml
          • /accumulo/trunk/fate/pom.xml
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
          • /accumulo/trunk/pom.xml
          • /accumulo/trunk/proxy/README
          • /accumulo/trunk/proxy/pom.xml
          • /accumulo/trunk/server
          • /accumulo/trunk/server/pom.xml
          • /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java
          • /accumulo/trunk/src
          • /accumulo/trunk/start/pom.xml
          • /accumulo/trunk/test/pom.xml
          • /accumulo/trunk/trace/pom.xml
          Show
          hudson Hudson added a comment - Integrated in Accumulo-Trunk #874 (See https://builds.apache.org/job/Accumulo-Trunk/874/ ) merge changes from 1.5 ACCUMULO-1421 simplify reflection lookup of SafeModeAction ACCUMULO-???? changed getConfiguration to be package protected so that it is not in the public API ACCUMULO-1274 classpath command shows two java configured classpaths before showing Accumulo configured classpaths. ACCUMULO-970 use reasonable defaults for batchwriter in documentation (Revision 1483510) ACCUMULO-1421 simplify reflection lookup of SafeModeAction (Revision 1483498) ACCUMULO-1421 , ACCUMULO-904 merge to trunk (Revision 1483461) ACCUMULO-1413 , ACCUMULO-1421 merge to trunk (Revision 1483431) Result = SUCCESS kturner : Files : /accumulo/trunk /accumulo/trunk/assemble /accumulo/trunk/core /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/analytics.tex /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/clients.tex /accumulo/trunk/examples /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java /accumulo/trunk/pom.xml /accumulo/trunk/proxy/README /accumulo/trunk/server /accumulo/trunk/src /accumulo/trunk/start/src/main/java/org/apache/accumulo/start/classloader/vfs/AccumuloVFSClassLoader.java /accumulo/trunk/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java /accumulo/trunk/test/src/test/java/org/apache/accumulo/test/ShellServerTest.java ecn : Files : /accumulo/trunk /accumulo/trunk/assemble /accumulo/trunk/core /accumulo/trunk/examples /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java /accumulo/trunk/pom.xml /accumulo/trunk/proxy/README /accumulo/trunk/server /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java /accumulo/trunk/src ctubbsii : Files : /accumulo/trunk /accumulo/trunk/assemble /accumulo/trunk/core /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/Property.java /accumulo/trunk/examples /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java /accumulo/trunk/pom.xml /accumulo/trunk/proxy/README /accumulo/trunk/server /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java /accumulo/trunk/src ctubbsii : Files : /accumulo/trunk /accumulo/trunk/README /accumulo/trunk/assemble /accumulo/trunk/assemble/pom.xml /accumulo/trunk/assemble/src/main/assemblies/binary-release.xml /accumulo/trunk/core /accumulo/trunk/core/pom.xml /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormat.java /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/TTimeoutTransport.java /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/administration.tex /accumulo/trunk/examples /accumulo/trunk/examples/pom.xml /accumulo/trunk/fate/pom.xml /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java /accumulo/trunk/pom.xml /accumulo/trunk/proxy/README /accumulo/trunk/proxy/pom.xml /accumulo/trunk/server /accumulo/trunk/server/pom.xml /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java /accumulo/trunk/src /accumulo/trunk/start/pom.xml /accumulo/trunk/test/pom.xml /accumulo/trunk/trace/pom.xml
          Hide
          hudson Hudson added a comment -

          Integrated in Accumulo-Trunk-Hadoop-2.0 #232 (See https://builds.apache.org/job/Accumulo-Trunk-Hadoop-2.0/232/)
          merge changes from 1.5

          ACCUMULO-1421 simplify reflection lookup of SafeModeAction
          ACCUMULO-???? changed getConfiguration to be package protected so that it is not in the public API
          ACCUMULO-1274 classpath command shows two java configured classpaths before showing Accumulo configured classpaths.
          ACCUMULO-970 use reasonable defaults for batchwriter in documentation (Revision 1483510)
          ACCUMULO-1421 simplify reflection lookup of SafeModeAction (Revision 1483498)
          ACCUMULO-1421, ACCUMULO-904 merge to trunk (Revision 1483461)
          ACCUMULO-1413,ACCUMULO-1421 merge to trunk (Revision 1483431)

          Result = FAILURE
          kturner :
          Files :

          • /accumulo/trunk
          • /accumulo/trunk/assemble
          • /accumulo/trunk/core
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          • /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/analytics.tex
          • /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/clients.tex
          • /accumulo/trunk/examples
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
          • /accumulo/trunk/pom.xml
          • /accumulo/trunk/proxy/README
          • /accumulo/trunk/server
          • /accumulo/trunk/src
          • /accumulo/trunk/start/src/main/java/org/apache/accumulo/start/classloader/vfs/AccumuloVFSClassLoader.java
          • /accumulo/trunk/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java
          • /accumulo/trunk/test/src/test/java/org/apache/accumulo/test/ShellServerTest.java

          ecn :
          Files :

          • /accumulo/trunk
          • /accumulo/trunk/assemble
          • /accumulo/trunk/core
          • /accumulo/trunk/examples
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
          • /accumulo/trunk/pom.xml
          • /accumulo/trunk/proxy/README
          • /accumulo/trunk/server
          • /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java
          • /accumulo/trunk/src

          ctubbsii :
          Files :

          • /accumulo/trunk
          • /accumulo/trunk/assemble
          • /accumulo/trunk/core
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/Property.java
          • /accumulo/trunk/examples
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
          • /accumulo/trunk/pom.xml
          • /accumulo/trunk/proxy/README
          • /accumulo/trunk/server
          • /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java
          • /accumulo/trunk/src

          ctubbsii :
          Files :

          • /accumulo/trunk
          • /accumulo/trunk/README
          • /accumulo/trunk/assemble
          • /accumulo/trunk/assemble/pom.xml
          • /accumulo/trunk/assemble/src/main/assemblies/binary-release.xml
          • /accumulo/trunk/core
          • /accumulo/trunk/core/pom.xml
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormat.java
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          • /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/TTimeoutTransport.java
          • /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/administration.tex
          • /accumulo/trunk/examples
          • /accumulo/trunk/examples/pom.xml
          • /accumulo/trunk/fate/pom.xml
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
          • /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
          • /accumulo/trunk/pom.xml
          • /accumulo/trunk/proxy/README
          • /accumulo/trunk/proxy/pom.xml
          • /accumulo/trunk/server
          • /accumulo/trunk/server/pom.xml
          • /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java
          • /accumulo/trunk/src
          • /accumulo/trunk/start/pom.xml
          • /accumulo/trunk/test/pom.xml
          • /accumulo/trunk/trace/pom.xml
          Show
          hudson Hudson added a comment - Integrated in Accumulo-Trunk-Hadoop-2.0 #232 (See https://builds.apache.org/job/Accumulo-Trunk-Hadoop-2.0/232/ ) merge changes from 1.5 ACCUMULO-1421 simplify reflection lookup of SafeModeAction ACCUMULO-???? changed getConfiguration to be package protected so that it is not in the public API ACCUMULO-1274 classpath command shows two java configured classpaths before showing Accumulo configured classpaths. ACCUMULO-970 use reasonable defaults for batchwriter in documentation (Revision 1483510) ACCUMULO-1421 simplify reflection lookup of SafeModeAction (Revision 1483498) ACCUMULO-1421 , ACCUMULO-904 merge to trunk (Revision 1483461) ACCUMULO-1413 , ACCUMULO-1421 merge to trunk (Revision 1483431) Result = FAILURE kturner : Files : /accumulo/trunk /accumulo/trunk/assemble /accumulo/trunk/core /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/analytics.tex /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/clients.tex /accumulo/trunk/examples /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java /accumulo/trunk/pom.xml /accumulo/trunk/proxy/README /accumulo/trunk/server /accumulo/trunk/src /accumulo/trunk/start/src/main/java/org/apache/accumulo/start/classloader/vfs/AccumuloVFSClassLoader.java /accumulo/trunk/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java /accumulo/trunk/test/src/test/java/org/apache/accumulo/test/ShellServerTest.java ecn : Files : /accumulo/trunk /accumulo/trunk/assemble /accumulo/trunk/core /accumulo/trunk/examples /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java /accumulo/trunk/pom.xml /accumulo/trunk/proxy/README /accumulo/trunk/server /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java /accumulo/trunk/src ctubbsii : Files : /accumulo/trunk /accumulo/trunk/assemble /accumulo/trunk/core /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/Property.java /accumulo/trunk/examples /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java /accumulo/trunk/pom.xml /accumulo/trunk/proxy/README /accumulo/trunk/server /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java /accumulo/trunk/src ctubbsii : Files : /accumulo/trunk /accumulo/trunk/README /accumulo/trunk/assemble /accumulo/trunk/assemble/pom.xml /accumulo/trunk/assemble/src/main/assemblies/binary-release.xml /accumulo/trunk/core /accumulo/trunk/core/pom.xml /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormat.java /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/TTimeoutTransport.java /accumulo/trunk/docs/src/main/latex/accumulo_user_manual/chapters/administration.tex /accumulo/trunk/examples /accumulo/trunk/examples/pom.xml /accumulo/trunk/fate/pom.xml /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java /accumulo/trunk/pom.xml /accumulo/trunk/proxy/README /accumulo/trunk/proxy/pom.xml /accumulo/trunk/server /accumulo/trunk/server/pom.xml /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/Accumulo.java /accumulo/trunk/src /accumulo/trunk/start/pom.xml /accumulo/trunk/test/pom.xml /accumulo/trunk/trace/pom.xml
          Hide
          kturner Keith Turner added a comment -

          Should we consider making a core.util.CompatibilityLayer

          I like the idea of having all of this crazy reflection code in one place. Do not really have a good reason. I just do not like the idea of it sprinkled throughout the codebase.

          Show
          kturner Keith Turner added a comment - Should we consider making a core.util.CompatibilityLayer I like the idea of having all of this crazy reflection code in one place. Do not really have a good reason. I just do not like the idea of it sprinkled throughout the codebase.
          Hide
          hudson Hudson added a comment -

          Integrated in Accumulo-1.5 #118 (See https://builds.apache.org/job/Accumulo-1.5/118/)
          ACCUMULO-1421 simplify reflection lookup of SafeModeAction, reverting inadvertent check-in of LargeRowTest (Revision 1483496)
          ACCUMULO-1421 simplify reflection lookup of SafeModeAction (Revision 1483494)
          ACCUMULO-1421 fix warnings introduced by previous commits (Revision 1483435)
          ACCUMULO-1421 added reflection to improve runtime compatibility between hadoop 1 and 2 JobContext objects (Revision 1483417)

          Result = SUCCESS
          ecn :
          Files :

          • /accumulo/branches/1.5/server/src/main/java/org/apache/accumulo/server/Accumulo.java
          • /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java

          ecn :
          Files :

          • /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java

          ctubbsii :
          Files :

          • /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          • /accumulo/branches/1.5/server/src/main/java/org/apache/accumulo/server/Accumulo.java

          afuchs :
          Files :

          • /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormat.java
          • /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java
          • /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          Show
          hudson Hudson added a comment - Integrated in Accumulo-1.5 #118 (See https://builds.apache.org/job/Accumulo-1.5/118/ ) ACCUMULO-1421 simplify reflection lookup of SafeModeAction, reverting inadvertent check-in of LargeRowTest (Revision 1483496) ACCUMULO-1421 simplify reflection lookup of SafeModeAction (Revision 1483494) ACCUMULO-1421 fix warnings introduced by previous commits (Revision 1483435) ACCUMULO-1421 added reflection to improve runtime compatibility between hadoop 1 and 2 JobContext objects (Revision 1483417) Result = SUCCESS ecn : Files : /accumulo/branches/1.5/server/src/main/java/org/apache/accumulo/server/Accumulo.java /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java ecn : Files : /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java ctubbsii : Files : /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java /accumulo/branches/1.5/server/src/main/java/org/apache/accumulo/server/Accumulo.java afuchs : Files : /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormat.java /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          Hide
          hudson Hudson added a comment -

          Integrated in Accumulo-1.5-Hadoop-2.0 #120 (See https://builds.apache.org/job/Accumulo-1.5-Hadoop-2.0/120/)
          ACCUMULO-1421 simplify reflection lookup of SafeModeAction, reverting inadvertent check-in of LargeRowTest (Revision 1483496)
          ACCUMULO-1421 simplify reflection lookup of SafeModeAction (Revision 1483494)
          ACCUMULO-1421 fix warnings introduced by previous commits (Revision 1483435)
          ACCUMULO-1421 added reflection to improve runtime compatibility between hadoop 1 and 2 JobContext objects (Revision 1483417)

          Result = SUCCESS
          ecn :
          Files :

          • /accumulo/branches/1.5/server/src/main/java/org/apache/accumulo/server/Accumulo.java
          • /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java

          ecn :
          Files :

          • /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java

          ctubbsii :
          Files :

          • /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          • /accumulo/branches/1.5/server/src/main/java/org/apache/accumulo/server/Accumulo.java

          afuchs :
          Files :

          • /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormat.java
          • /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java
          • /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          Show
          hudson Hudson added a comment - Integrated in Accumulo-1.5-Hadoop-2.0 #120 (See https://builds.apache.org/job/Accumulo-1.5-Hadoop-2.0/120/ ) ACCUMULO-1421 simplify reflection lookup of SafeModeAction, reverting inadvertent check-in of LargeRowTest (Revision 1483496) ACCUMULO-1421 simplify reflection lookup of SafeModeAction (Revision 1483494) ACCUMULO-1421 fix warnings introduced by previous commits (Revision 1483435) ACCUMULO-1421 added reflection to improve runtime compatibility between hadoop 1 and 2 JobContext objects (Revision 1483417) Result = SUCCESS ecn : Files : /accumulo/branches/1.5/server/src/main/java/org/apache/accumulo/server/Accumulo.java /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java ecn : Files : /accumulo/branches/1.5/test/src/main/java/org/apache/accumulo/test/functional/LargeRowTest.java ctubbsii : Files : /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java /accumulo/branches/1.5/server/src/main/java/org/apache/accumulo/server/Accumulo.java afuchs : Files : /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormat.java /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java /accumulo/branches/1.5/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
          Hide
          afuchs Adam Fuchs added a comment -

          RE location of getConfiguration: how about if we make it package protected in InputFormatBase for now?

          RE compatibility layer: As a general solution I would say that's a good solution, but since we can (in theory) enumerate all of the versions we want to be compatible with it's better if we can just make one simple release to rule them all.

          RE 1.5 vs. trunk: Is there another point of incompatibility? If this just fixes the problem entirely then we definitely want to have it in 1.5.0. If not, then we haven't sacrificed much by putting it in 1.5. Sean, are you running compatibility tests with CDH 3 and 4?

          Show
          afuchs Adam Fuchs added a comment - RE location of getConfiguration: how about if we make it package protected in InputFormatBase for now? RE compatibility layer: As a general solution I would say that's a good solution, but since we can (in theory) enumerate all of the versions we want to be compatible with it's better if we can just make one simple release to rule them all. RE 1.5 vs. trunk: Is there another point of incompatibility? If this just fixes the problem entirely then we definitely want to have it in 1.5.0. If not, then we haven't sacrificed much by putting it in 1.5. Sean, are you running compatibility tests with CDH 3 and 4?
          Hide
          busbey Sean Busbey added a comment -

          A compatibility layer would be a great idea. It also gives some room for dealing with future incompatibilities. Hive (HIVE-3029) does this, and Pig (PIG-2686) used to. Probably a package, as Hive does, would be best.

          I think pushing for trunk is prudent, since the pressure to get 1.5 out is high and there's uncertainty on how wide a change this needs.

          Show
          busbey Sean Busbey added a comment - A compatibility layer would be a great idea. It also gives some room for dealing with future incompatibilities. Hive ( HIVE-3029 ) does this, and Pig ( PIG-2686 ) used to. Probably a package, as Hive does, would be best. I think pushing for trunk is prudent, since the pressure to get 1.5 out is high and there's uncertainty on how wide a change this needs.
          Hide
          billie.rinaldi Billie Rinaldi added a comment -

          Putting the getConfiguration(JobContext) method in InputFormatBase makes it part of the public API. Should we consider making a core.util.CompatibilityLayer class where we could put stuff like this, and then it will be easy to get rid of it when we no longer need it?

          Do we want this in 1.5 yet, or just trunk? It will only be needed when we've tracked down all the incompatibilities. And if their fixes are all non-API-changing like this, it will be an easy decision to merge them to 1.5 given the benefits. On the other hand ... if it's possible that we might only support hadoop 2 for 1.6, then we only want this fix in 1.5. I guess there's no way to predict that at this point.

          Show
          billie.rinaldi Billie Rinaldi added a comment - Putting the getConfiguration(JobContext) method in InputFormatBase makes it part of the public API. Should we consider making a core.util.CompatibilityLayer class where we could put stuff like this, and then it will be easy to get rid of it when we no longer need it? Do we want this in 1.5 yet, or just trunk? It will only be needed when we've tracked down all the incompatibilities. And if their fixes are all non-API-changing like this, it will be an easy decision to merge them to 1.5 given the benefits. On the other hand ... if it's possible that we might only support hadoop 2 for 1.6, then we only want this fix in 1.5. I guess there's no way to predict that at this point.
          Hide
          busbey Sean Busbey added a comment -

          Oh I see, I can just do both myself. Fair point.

          Show
          busbey Sean Busbey added a comment - Oh I see, I can just do both myself. Fair point.
          Hide
          busbey Sean Busbey added a comment -

          The version is very relevant. I need to test on the opposite one to make sure we haven't missed references to incompatible classes.

          I guess I can just search through the byte code. :/

          Show
          busbey Sean Busbey added a comment - The version is very relevant. I need to test on the opposite one to make sure we haven't missed references to incompatible classes. I guess I can just search through the byte code. :/
          Hide
          vines John Vines added a comment -

          I believe the version is irrelevant.

          Sent from my phone, please pardon the typos and brevity.

          Show
          vines John Vines added a comment - I believe the version is irrelevant. Sent from my phone, please pardon the typos and brevity.
          Hide
          busbey Sean Busbey added a comment -

          If I wanted to test this, which profile did you build with?

          Show
          busbey Sean Busbey added a comment - If I wanted to test this, which profile did you build with?

            People

            • Assignee:
              afuchs Adam Fuchs
              Reporter:
              afuchs Adam Fuchs
            • Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development