Skip to content

Commit

Permalink
[SPARK-3955] Different versions between jackson-mapper-asl and jackso…
Browse files Browse the repository at this point in the history
…n-c...

...ore-asl

- set the same version to jackson-mapper-asl and jackson-core-asl
- It's related with #2818
- coded a same patch from a latest master

Author: Jongyoul Lee <jongyoul@gmail.com>

Closes #3716 from jongyoul/SPARK-3955 and squashes the following commits:

efa29aa [Jongyoul Lee] [SPARK-3955] Different versions between jackson-mapper-asl and jackson-core-asl - set the same version to jackson-mapper-asl and jackson-core-asl
  • Loading branch information
jongyoul authored and pwendell committed Dec 27, 2014
1 parent 82bf4be commit 2483c1e
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -149,6 +149,7 @@
<scala.binary.version>2.10</scala.binary.version>
<jline.version>${scala.version}</jline.version>
<jline.groupid>org.scala-lang</jline.groupid>
<jackson.version>1.8.8</jackson.version>
</properties>

<repositories>
Expand Down Expand Up @@ -819,10 +820,15 @@
</exclusions>
</dependency>
<dependency>
<!-- Matches the version of jackson-core-asl pulled in by avro -->
<!-- Matches the versions of jackson-mapper-asl and jackson-core-asl with avro -->
<groupId>org.codehaus.jackson</groupId>
<artifactId>jackson-mapper-asl</artifactId>
<version>1.8.8</version>
<version>${jackson.version}</version>
</dependency>
<dependency>
<groupId>org.codehaus.jackson</groupId>
<artifactId>jackson-mapper-asl</artifactId>
<version>${jackson.version}</version>
</dependency>
</dependencies>
</dependencyManagement>
Expand Down

2 comments on commit 2483c1e

@witgo
Copy link
Contributor

@witgo witgo commented on 2483c1e Dec 28, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code is the same as the above?

      <dependency>
        <!-- Matches the versions of jackson-mapper-asl and jackson-core-asl with avro -->
        <groupId>org.codehaus.jackson</groupId>
        <artifactId>jackson-mapper-asl</artifactId>
        <version>${jackson.version}</version>
      </dependency>
      <dependency>
        <groupId>org.codehaus.jackson</groupId>
        <artifactId>jackson-mapper-asl</artifactId>
        <version>${jackson.version}</version>
      </dependency>

@jongyoul
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@witgo This has been resolved by another PR. Thx.

Please sign in to comment.