Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-7461

Jackson Dependency Not Declared in Hadoop POM

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.20.205.0
    • Fix Version/s: 1.0.0
    • Component/s: build
    • Labels:
      None
    • Target Version/s:

      Description

      (COMMENT: This bug still affects 0.20.205.0, four months after the bug was filed. This causes total failure, and the fix is trivial for whoever manages the POM – just add the missing dependency! --ben)

      This issue was identified and the fix & workaround was documented at

      https://issues.cloudera.org/browse/DISTRO-44

      The issue affects use of Hadoop 0.20.203.0 from the Maven central repo. I built a job using that maven repo and ran it, resulting in this failure:

      Exception in thread "main" java.lang.NoClassDefFoundError: org/codehaus/jackson/map/JsonMappingException
      at thinkbig.hadoop.inputformat.TestXmlInputFormat.run(TestXmlInputFormat.java:18)
      at thinkbig.hadoop.inputformat.TestXmlInputFormat.main(TestXmlInputFormat.java:23)
      Caused by: java.lang.ClassNotFoundException: org.codehaus.jackson.map.JsonMappingException

      1. HADOOP-7461-1.patch
        0.5 kB
        Giridharan Kesavan
      2. HADOOP-7461.patch
        0.5 kB
        Giridharan Kesavan

        Activity

        Hide
        mattf Matt Foley added a comment -

        This was indeed fixed in 1.0.0, so no need to note it as fixed in 1.1.0.

        Show
        mattf Matt Foley added a comment - This was indeed fixed in 1.0.0, so no need to note it as fixed in 1.1.0.
        Hide
        mattf Matt Foley added a comment -

        Closed upon release of version 1.0.0.

        Show
        mattf Matt Foley added a comment - Closed upon release of version 1.0.0.
        Hide
        gkesavan Giridharan Kesavan added a comment -

        committed to both 1 and 1.0 branches, thanks Matt

        Show
        gkesavan Giridharan Kesavan added a comment - committed to both 1 and 1.0 branches, thanks Matt
        Hide
        mattf Matt Foley added a comment -

        +1. Please commit to branch-1.0 and branch-1. Thanks.

        Show
        mattf Matt Foley added a comment - +1. Please commit to branch-1.0 and branch-1. Thanks.
        Hide
        gkesavan Giridharan Kesavan added a comment -

        mybad , fixed the typos.

        tested the patch by doing mvn-install and verified if the pom includes the jackson dependency.

        steps:
        ant -Dcompile.native=true -Dcompile.c++=true mvn-install
        check the mvn cache
        ~/.m2/repository/org/apache/hadoop/hadoop-core/1.0.1-SNAPSHOT/hadoop-core-1.0.1-SNAPSHOT.pom

        Show
        gkesavan Giridharan Kesavan added a comment - mybad , fixed the typos. tested the patch by doing mvn-install and verified if the pom includes the jackson dependency. steps: ant -Dcompile.native=true -Dcompile.c++=true mvn-install check the mvn cache ~/.m2/repository/org/apache/hadoop/hadoop-core/1.0.1-SNAPSHOT/hadoop-core-1.0.1-SNAPSHOT.pom
        Hide
        mattf Matt Foley added a comment -

        Giri, two typos:
        + <artifactId>jackson-mapper-asl</groupId>
        should be
        + <artifactId>jackson-mapper-asl</artifactId>

        and the closing
        + <dependency>
        should be
        + </dependency>

        Show
        mattf Matt Foley added a comment - Giri, two typos: + <artifactId>jackson-mapper-asl</groupId> should be + <artifactId>jackson-mapper-asl</artifactId> and the closing + <dependency> should be + </dependency>
        Hide
        mattf Matt Foley added a comment -

        Two comments:
        First, to request/propose that a bug be fixed in a given release, please add that release to the "Target Version" field in the bug report.
        Second, if the fix is trivial, please construct a patch for the fix, test it, and upload it to the Jira, rather than complaining that no one (else) has fixed it in four months.

        Show
        mattf Matt Foley added a comment - Two comments: First, to request/propose that a bug be fixed in a given release, please add that release to the "Target Version" field in the bug report. Second, if the fix is trivial, please construct a patch for the fix, test it, and upload it to the Jira, rather than complaining that no one (else) has fixed it in four months.
        Hide
        gkesavan Giridharan Kesavan added a comment -

        updated the core pom template to include the jackson dependency.

        Show
        gkesavan Giridharan Kesavan added a comment - updated the core pom template to include the jackson dependency.
        Hide
        benwing Ben Wing added a comment -

        I've upgraded this to Major in the hope that someone will pay attention. This bug still affects 0.20.205.0, four months after the bug was filed. It causes total failure when running in non-distributed mode. The fix is trivial for whoever manages the POM – just add the missing dependency! It's a bad sign that a bug like this has persisted this long.

        Show
        benwing Ben Wing added a comment - I've upgraded this to Major in the hope that someone will pay attention. This bug still affects 0.20.205.0, four months after the bug was filed. It causes total failure when running in non-distributed mode. The fix is trivial for whoever manages the POM – just add the missing dependency! It's a bad sign that a bug like this has persisted this long.

          People

          • Assignee:
            gkesavan Giridharan Kesavan
            Reporter:
            rbodkin Ron Bodkin
          • Votes:
            2 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development