[ 
https://issues.apache.org/jira/browse/AVRO-1126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673641#comment-16673641
 ] 

Doug Cutting commented on AVRO-1126:
------------------------------------

I wonder if this is the time to also remove the (deprecated) places where 
Avro's API leaks references to Jackson classes?

 

JsonProperties has a number of such deprecated methods, and Json.Reader and 
Json.Writer are classes deprecated for this reason.  
Schema.addProp(String,JsonNode) should have been deprecated, since 
Schema.addProp(String,Object) is preferred.  Schema.parse(JsonNode) was 
correctly deprecated.

Code that uses these will need to be updated after this change, so we might as 
well have them update to using the Object representation of json data, no?

In general, Avro's public APIs should no longer reference Jackson classes.  
Jackson should only be used internally.

> Upgrade to Jackson 2+
> ---------------------
>
>                 Key: AVRO-1126
>                 URL: https://issues.apache.org/jira/browse/AVRO-1126
>             Project: Avro
>          Issue Type: Task
>          Components: java
>            Reporter: James Tyrrell
>            Assignee: Charles Honton
>            Priority: Critical
>             Fix For: 1.9.0
>
>
> Quite annoyingly with Jackson 2+ the base package name has changed from 
> org.codehaus.jackson to com.fasterxml.jackson so in addition to changing the 
> dependencies from:
> {code:xml} 
> <dependency>
>     <groupId>org.codehaus.jackson</groupId>
>     <artifactId>jackson-core-asl</artifactId>
>     <version>${jackson.version}</version>
> </dependency>
> <dependency>
>     <groupId>org.codehaus.jackson</groupId>
>     <artifactId>jackson-mapper-asl</artifactId>
>     <version>${jackson.version}</version>
> </dependency>
> {code} 
> to:
> {code:xml} 
> <dependency>
>     <groupId>com.fasterxml.jackson.core</groupId>
>     <artifactId>jackson-core</artifactId>
>     <version>${jackson.version}</version>
> </dependency>
> <dependency>
>     <groupId>com.fasterxml.jackson.core</groupId>
>     <artifactId>jackson-databind</artifactId>
>     <version>${jackson.version}</version>
> </dependency>
> {code} 
> the base package in the code needs to be updated. More info can be found 
> [here|http://wiki.fasterxml.com/JacksonUpgradeFrom19To20], I am happy to do 
> the work just let me know what is preferable i.e. should I just attach a 
> patch to this issue?



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to