You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@helix.apache.org by GitBox <gi...@apache.org> on 2021/02/26 04:39:05 UTC

[GitHub] [helix] pkuwm opened a new issue #1655: Jackson 1.x serializer does not work with ZNRecord

pkuwm opened a new issue #1655:
URL: https://github.com/apache/helix/issues/1655


   ### Description
   "org.codehaus.jackson:jackson-mapper-asl:1.9.13" because 1.9.13 can not deserialize the bytes when used by helix client.
   
   ```
   org.codehaus.jackson.map.JsonMappingException: Type variable 'T' can not be resolved (with context of class org.apache.helix.zookeeper.datamodel.ZNRecord)
           at org.codehaus.jackson.map.deser.StdDeserializerProvider._createAndCache2(StdDeserializerProvider.java:315) ~[jackson-mapper-asl-1.9.13.jar:1.9.13]
           at org.codehaus.jackson.map.deser.StdDeserializerProvider._createAndCacheValueDeserializer(StdDeserializerProvider.java:290) ~[jackson-mapper-asl-1.9.13.jar:1.9.13]
           at org.codehaus.jackson.map.deser.StdDeserializerProvider.findValueDeserializer(StdDeserializerProvider.java:159) ~[jackson-mapper-asl-1.9.13.jar:1.9.13]
           at org.codehaus.jackson.map.deser.StdDeserializerProvider.findTypedValueDeserializer(StdDeserializerProvider.java:180) ~[jackson-mapper-asl-1.9.13.jar:1.9.13]
           at org.codehaus.jackson.map.ObjectMapper._findRootDeserializer(ObjectMapper.java:2829) ~[jackson-mapper-asl-1.9.13.jar:1.9.13]
           at org.codehaus.jackson.map.ObjectMapper._readMapAndClose(ObjectMapper.java:2728) ~[jackson-mapper-asl-1.9.13.jar:1.9.13]
           at org.codehaus.jackson.map.ObjectMapper.readValue(ObjectMapper.java:1909) ~[jackson-mapper-asl-1.9.13.jar:1.9.13]
           at com.abc.PropertyCompressedJsonSerializer.deserialize(PropertyCompressedJsonSerializer.java:81) []
           at org.apache.helix.zookeeper.zkclient.serialize.BasicZkSerializer.deserialize(BasicZkSerializer.java:39) [helix-core-1.0.1.2402.jar:1.0.1.2402]
           at org.apache.helix.zookeeper.zkclient.ZkClient.deserialize(ZkClient.java:1791) [helix-core-1.0.1.2402.jar:1.0.1.2402]
           at org.apache.helix.zookeeper.zkclient.ZkClient.readData(ZkClient.java:1830) [helix-core-1.0.1.2402.jar:1.0.1.2402]
           at org.apache.helix.zookeeper.zkclient.ZkClient.readData(ZkClient.java:1814) [helix-core-1.0.1.2402.jar:1.0.1.2402]
           at org.apache.helix.manager.zk.ZkBaseDataAccessor.get(ZkBaseDataAccessor.java:472) [helix-core-1.0.1.2402.jar:1.0.1.2402]
           at org.apache.helix.manager.zk.ZkCacheBaseDataAccessor.get(ZkCacheBaseDataAccessor.java:391) [helix-core-1.0.1.2402.jar:1.0.1.2402]
   ```
   
   ### Root cause
   Jackson was upgraded in helix from 1.x to 2.x. Jackson 1.x does not support Jackson 2.x annotations.
   
   ### Proposed solution
   According to Jackson's recommendation on GitHub doc: https://github.com/FasterXML/jackson-annotations,
   Backwards compatibility:
   
   - You can make Jackson 2 use Jackson 1 annotations with [jackson-legacy-introspector](https://github.com/Laures/jackson-legacy-introspector)
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org


[GitHub] [helix] pkuwm closed issue #1655: Jackson 1.x serializer does not work with ZNRecord

Posted by GitBox <gi...@apache.org>.
pkuwm closed issue #1655:
URL: https://github.com/apache/helix/issues/1655


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org


[GitHub] [helix] narendly commented on issue #1655: Jackson 1.x serializer does not work with ZNRecord

Posted by GitBox <gi...@apache.org>.
narendly commented on issue #1655:
URL: https://github.com/apache/helix/issues/1655#issuecomment-786788137


   Good find!


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org