[ https://issues.apache.org/jira/browse/AMQ-7072?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16649115#comment-16649115 ]
ASF GitHub Bot commented on AMQ-7072: ------------------------------------- GitHub user rmannibucau opened a pull request: https://github.com/apache/activemq/pull/308 [AMQ-7072] moving to JSON-B API instead of jackson Goal is to let users switch the impl on need + using johnzon as default Issue was identified on TomEE where jackson is not desired in the distribution cause there is already johnzon and some You can merge this pull request into a Git repository by running: $ git pull https://github.com/rmannibucau/activemq AMQ-7072_jsonb-instead-of-jackson Alternatively you can review and apply these changes as the patch at: https://github.com/apache/activemq/pull/308.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #308 ---- commit dabdf20b29567cb27505b15ad7c3d105ec8f2063 Author: Romain Manni-Bucau <rmannibucau@...> Date: 2018-10-13T20:40:06Z AMQ-7072 moving to JSON-B API instead of jackson to let users switch the impl on need + using johnzon as default ---- > ActiveMQ shouldn't import jackson but use JSON-B instead of jackson to > support impl switch > ------------------------------------------------------------------------------------------ > > Key: AMQ-7072 > URL: https://issues.apache.org/jira/browse/AMQ-7072 > Project: ActiveMQ > Issue Type: Improvement > Reporter: Romain Manni-Bucau > Assignee: Jean-Baptiste Onofré > Priority: Major > > The regression we hit at the moment is that activemq enforces TomEE to import > jackson whereas it wants to keep johnzon as JSON mapper impl. Since JSON-B > spec is out and implemented by both I guess it can be the way to solve that > issue. > The most blocking thing is > ./activemq-broker/src/main/java/org/apache/activemq/broker/jmx/DestinationsViewFilter.java > - which can already not create a mapper if json is empty ;) - but here is > the list of code location which would be neat to fix: > {code} > ./activemq-partition/src/main/java/org/apache/activemq/partition/dto/Partitioning.java:import > com.fasterxml.jackson.annotation.JsonInclude; > ./activemq-partition/src/main/java/org/apache/activemq/partition/dto/Partitioning.java:import > com.fasterxml.jackson.databind.DeserializationFeature; > ./activemq-partition/src/main/java/org/apache/activemq/partition/dto/Partitioning.java:import > com.fasterxml.jackson.databind.ObjectMapper; > ./activemq-partition/src/main/java/org/apache/activemq/partition/dto/Partitioning.java:import > com.fasterxml.jackson.databind.DeserializationConfig; > ./activemq-partition/src/main/java/org/apache/activemq/partition/dto/Partitioning.java:import > com.fasterxml.jackson.databind.SerializationFeature; > ./activemq-partition/src/main/java/org/apache/activemq/partition/dto/Partitioning.java:import > com.fasterxml.jackson.databind.annotation.JsonDeserialize; > ./activemq-partition/src/main/java/org/apache/activemq/partition/dto/Partitioning.java:import > com.fasterxml.jackson.databind.annotation.JsonSerialize; > ./activemq-partition/src/main/java/org/apache/activemq/partition/dto/Partitioning.java:import > com.fasterxml.jackson.annotation.JsonProperty; > ./activemq-partition/src/main/java/org/apache/activemq/partition/dto/Target.java:import > com.fasterxml.jackson.annotation.JsonProperty; > ./activemq-broker/src/main/java/org/apache/activemq/broker/jmx/DestinationsViewFilter.java:import > com.fasterxml.jackson.databind.ObjectMapper; > ./activemq-leveldb-store/src/main/java/org/apache/activemq/leveldb/replicated/dto/LogWrite.java:import > com.fasterxml.jackson.annotation.JsonIgnoreProperties; > ./activemq-leveldb-store/src/main/java/org/apache/activemq/leveldb/replicated/dto/WalAck.java:import > com.fasterxml.jackson.annotation.JsonIgnoreProperties; > ./activemq-leveldb-store/src/main/java/org/apache/activemq/leveldb/replicated/dto/LogDelete.java:import > com.fasterxml.jackson.annotation.JsonIgnoreProperties; > ./activemq-leveldb-store/src/main/java/org/apache/activemq/leveldb/replicated/dto/Transfer.java:import > com.fasterxml.jackson.annotation.JsonIgnoreProperties; > ./activemq-leveldb-store/src/main/java/org/apache/activemq/leveldb/replicated/dto/Login.java:import > com.fasterxml.jackson.annotation.JsonIgnoreProperties; > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)