aldettinger commented on a change in pull request #2972:
URL: https://github.com/apache/camel-quarkus/pull/2972#discussion_r681561954
##########
File path:
extensions/jolt/deployment/src/main/java/org/apache/camel/quarkus/component/jolt/deployment/JoltProcessor.java
##########
@@ -36,5 +36,15 @@ void
registerReflectiveClasses(BuildProducer<ReflectiveClassBuildItem> producer)
ChainrEntry.STOCK_TRANSFORMS.values().stream().forEach(c -> {
producer.produce(new ReflectiveClassBuildItem(false, false, c));
});
+
+ // A JOLT Defaultr transformation spec is a JSON content and it needs
to be serialized at some point.
+ // As such, we need to register all JSON base types and super types
for serialization.
+ producer.produce(new ReflectiveClassBuildItem(false, false, false,
true, java.lang.Boolean.class.getName()));
+ producer.produce(new ReflectiveClassBuildItem(false, false, false,
true, java.lang.Number.class.getName()));
+ producer.produce(new ReflectiveClassBuildItem(false, false, false,
true, java.lang.Double.class.getName()));
+ producer.produce(new ReflectiveClassBuildItem(false, false, false,
true, java.lang.Integer.class.getName()));
+ producer.produce(new ReflectiveClassBuildItem(false, false, false,
true, java.lang.String.class.getName()));
+ producer.produce(new ReflectiveClassBuildItem(false, false, false,
true, java.util.AbstractList.class.getName()));
+ producer.produce(new ReflectiveClassBuildItem(false, false, false,
true, java.util.ArrayList.class.getName()));
Review comment:
Make sense to consider JSON types as common for camel. I've pushed an
update in this direction.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
